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

Auto license check gh action #5094

Merged
merged 8 commits into from
Jul 26, 2022
Merged

Auto license check gh action #5094

merged 8 commits into from
Jul 26, 2022

Conversation

antgamdia
Copy link
Contributor

Description of the change

This PR adds a gh action to check the copyright headers are present in all the files we add/edit in each pull request.

Benefits

All of our files will have the expected license headers.

Possible drawbacks

N/A

Applicable issues

Additional information

image

Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
@netlify
Copy link

netlify bot commented Jul 17, 2022

Deploy Preview for kubeapps-dev canceled.

Name Link
🔨 Latest commit 88297eb
🔍 Latest deploy log https://app.netlify.com/sites/kubeapps-dev/deploys/62dfcede8d35a500097f45af

Copy link
Collaborator

@castelblanque castelblanque left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>

Conflicts:
	cmd/kubeapps-apis/plugins/fluxv2/packages/v1alpha1/testdata/registry-app.yaml
@antgamdia antgamdia merged commit c50dca2 into main Jul 26, 2022
@antgamdia antgamdia deleted the 4426-autoLicenseCheck branch July 26, 2022 14:43
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.

Add CI step to check that all files contain copyright header
3 participants