-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
CI check for file-header added #2178
CI check for file-header added #2178
Conversation
74deb33
to
f6d15d6
Compare
Thanks for adding this. Looks like a couple of files are missing the header :) |
f6d15d6
to
3c4a026
Compare
Signed-off-by: Kunal Kushwaha <kushwaha_kunal_v7@lab.ntt.co.jp>
Updated missing header of new files. The CI error is due to blank lines at EOF in |
You should be able to add that to the ignore var like the vendor scripts are added |
12725bf
to
f040e6a
Compare
file-header check using github.com/kunalkushwaha/ltag Signed-off-by: Kunal Kushwaha <kushwaha_kunal_v7@lab.ntt.co.jp>
CI error will be fixed with merge of vbatts/git-validation#34 |
@crosbymichael This needs re-tun the travis. vbatts/git-validation#34 is merged, so it will pass the CI test. |
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.
LGTM
LGTM |
file-header check using github.com/kunalkushwaha/ltag
follow-up of #2108
Signed-off-by: Kunal Kushwaha kushwaha_kunal_v7@lab.ntt.co.jp