-
Notifications
You must be signed in to change notification settings - Fork 1.5k
consistent copyright notice #14531
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
consistent copyright notice #14531
Conversation
Reviewer's Guide by SourceryThis pull request updates the copyright notices in various files across the repository to 'The KubeVirt Authors', replacing the previous 'Red Hat, Inc.' attribution. The change was implemented using No diagrams generated as the changes look simple and do not need a visual representation. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @dankenigsberg - I've reviewed your changes - here's some feedback:
Overall Comments:
- Consider adding a script to update the copyright headers automatically in the future.
Here's what I looked at during the review
- 🟢 General issues: all looks good
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Red Hat is willing to mark these files as copyrighted by the KubeVirt Authors. This is a follow-up of commit e36bb0b sed -i 's/\(20.., \)*20.. Red Hat, Inc./The KubeVirt Authors./' `git grep -l '20.. Red Hat, Inc.'|grep -v staging|grep -v vendor|grep -v kubevirtci` Signed-off-by: Dan Kenigsberg <danken@redhat.com>
013826b
to
11c4702
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks
basically this one is enough in this case i think, but it doesnt matter much
(of course the current is more robust the one below is just simpler to this case)
sed -i 's/20.. Red Hat, Inc./The KubeVirt Authors./' git grep -l '20.. Red Hat, Inc.'|grep -v staging|grep -v vendor
Pull requests that are marked with After that period the bot marks them with the label /label needs-approver-review |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: enp0s3 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Required labels detected, running phase 2 presubmits: |
/remove-label needs-approver-review |
Red Hat is willing to mark these files as copyrighted by the KubeVirt Authors. This is a follow-up of commit e36bb0b
sed -i 's/(20.., )*20.. Red Hat, Inc./The KubeVirt Authors./'
git grep -l '20.. Red Hat, Inc.'|grep -v staging|grep -v vendor
/kind cleanup