Skip to content
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

Fixes #19534: All .cf and .st files under in rudder-techniques should have an updated license header #1681

Conversation

Fdall
Copy link
Contributor

@Fdall Fdall commented Jul 13, 2021

qa-test Outdated
{
find . -name '*.cf' -print0 |
while IFS= read -r -d '' file; do
if ! grep -qE "# Copyright 20[0-9]{2} Normation SAS" $file; then
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2100 limit seems fair

@Fdall
Copy link
Contributor Author

Fdall commented Jul 15, 2021

PR updated with a new commit

@Fdall
Copy link
Contributor Author

Fdall commented Jul 15, 2021

PR updated with a new commit

@Fdall
Copy link
Contributor Author

Fdall commented Jul 15, 2021

PR updated with a new commit

@Fdall
Copy link
Contributor Author

Fdall commented Jul 15, 2021

PR rebased

@Fdall Fdall force-pushed the ust_19534/all_cf_and_st_files_under_in_rudder_techniques_should_have_an_updated_license_header branch from 856a346 to c04e1ae Compare July 15, 2021 14:10
@Fdall
Copy link
Contributor Author

Fdall commented Jul 15, 2021

PR updated with a new commit

@Fdall
Copy link
Contributor Author

Fdall commented Jul 15, 2021

PR updated with a new commit

@Fdall
Copy link
Contributor Author

Fdall commented Jul 15, 2021

PR updated with a new commit

@Normation-Quality-Assistant
Copy link
Contributor

This PR is not mergeable to upper versions.
Since it is "Ready for merge" you must merge it by yourself using the following command:
rudder-dev merge https://github.com/Normation/rudder-techniques/pull/1681
-- Your faithful QA
Kant merge: "In law a man is guilty when he violates the rights of others. In ethics he is guilty if he only thinks of doing so."
(https://ci.normation.com/jenkins/job/merge-accepted-pr/41473/console)

@Fdall
Copy link
Contributor Author

Fdall commented Aug 25, 2021

OK, squash merging this PR

@Fdall Fdall force-pushed the ust_19534/all_cf_and_st_files_under_in_rudder_techniques_should_have_an_updated_license_header branch from 9b73cf6 to e8aa9db Compare August 25, 2021 14:32
@Fdall Fdall merged commit e8aa9db into Normation:branches/rudder/6.2 Aug 25, 2021
@Fdall
Copy link
Contributor Author

Fdall commented Aug 25, 2021

OK, merging this PR

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants