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

doc: Insert Arista license header in all source files #367

Merged

Conversation

carlbuchmann
Copy link
Contributor

@carlbuchmann carlbuchmann commented Aug 17, 2023

Change Summary

Insert license header in all source files

Proposed changes

Add pre-commit hook to insert license headers in:

  • Python files
  • Documentation files

Increase the scope of pre-commit hooks to include ./docs

The following files had additional fixes from pre-commit hooks

trim trailing whitespace.......................................................Failed
- hook id: trailing-whitespace
- exit code: 1
- files were modified by this hook

Fixing docs/advanced_usages/as-python-lib.md
Fixing docs/cli/get-inventory-information.md
Fixing docs/getting-started.md
Fixing docs/cli/debug.md

fix end of files...............................................................Failed
- hook id: end-of-file-fixer
- exit code: 1
- files were modified by this hook

Fixing docs/imgs/animated-svg.md
Fixing docs/snippets/anta_help.txt
Fixing docs/api/tests.system.md
Fixing docs/imgs/anta-nrfu.svg
Fixing docs/api/tests.md
Fixing docs/faq.md
Fixing docs/imgs/anta-nrfu.cast

@titom73 titom73 changed the title Doc: Insert license header in all source files doc: Insert Arista license header in all source files Aug 18, 2023
@titom73 titom73 added the refactor Code refactoring label Aug 18, 2023
license-short.txt Outdated Show resolved Hide resolved
@carlbuchmann carlbuchmann changed the title doc: Insert Arista license header in all source files Doc: Insert Arista license header in all source files Aug 18, 2023
@carlbuchmann carlbuchmann changed the title Doc: Insert Arista license header in all source files doc: Insert Arista license header in all source files Aug 18, 2023
@carlbuchmann carlbuchmann marked this pull request as ready for review August 18, 2023 12:48
@carlbuchmann carlbuchmann marked this pull request as draft August 18, 2023 12:48
@carlbuchmann carlbuchmann marked this pull request as ready for review August 18, 2023 13:28
@titom73 titom73 added the CI label Aug 18, 2023
Copy link
Collaborator

@titom73 titom73 left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you

@titom73 titom73 added this to the v0.8.0 milestone Aug 18, 2023
@titom73 titom73 merged commit 0065d6a into arista-netdevops-community:main Aug 18, 2023
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants