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.

Sign up for GitHub

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

Jump to bottom

Doc: Insert license header in all source files #664

Merged
merged 3 commits into from
Nov 21, 2023
Merged

Doc: Insert license header in all source files #664

merged 3 commits into from
Nov 21, 2023

Conversation

carlbuchmann
Copy link
Member

@carlbuchmann carlbuchmann commented Aug 18, 2023 •
edited
Loading

Change Summary

Insert license header in all source files

Proposed changes

Add pre-commit hook to insert license headers in:

  • Python files
  • Jinja2 templates
  • Documentation files
  • yaml ansible tasks and handlers

Repository Checklist

  • My code has been rebased from devel before I start
  • I have read the CONTRIBUTING document.
  • My change requires a change to the documentation and documentation have been updated accordingly. (check the box if not applicable)
  • I have updated molecule CI testing accordingly. (check the box if not applicable)

@github-actions github-actions bot requested a review from sugetha24 August 18, 2023 12:06
@github-actions github-actions bot added data-model changes module: cv_configlet Issue related to cv_configlet module module: cv_configlet_v3 Issue related to cv_cononfiglet module in v3 module: cv_container Issue related to cv_container module module: cv_container_v3 Issue related to cv_container module in v3 module: cv_device Issue related to cv_device module module: cv_device_v3 Issue related to cv_device module in v3 module: cv_facts Issue related to cv_facts module module: cv_facts_v3 module: cv_task Issue related to cv_task module module: cv_task_v3 Issue related to cv_tasks module in v3 module_utils: cv_client cv_client implementation issue role: configlets_sync Issues related to configlets_sync role role: ztp_configuration Issue related to ztp_configuration role type: documentation Improvements or additions to documentation labels Aug 18, 2023
@sugetha24 sugetha24 modified the milestones: v3.8.0, v3.9.0 Sep 7, 2023
@sugetha24 sugetha24 modified the milestones: v3.9.0, v3.8.0 Nov 1, 2023
@carlbuchmann carlbuchmann marked this pull request as ready for review November 20, 2023 20:23
@github-actions github-actions bot added the state: conflict PR has conflict to resolve label Nov 20, 2023
@github-actions GitHub Actions
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions GitHub Actions
Copy link

Conflicts have been resolved. A maintainer will review the pull request shortly.

@github-actions github-actions bot removed the state: conflict PR has conflict to resolve label Nov 21, 2023
@sugetha24 sugetha24 merged commit cf34b42 into aristanetworks:devel Nov 21, 2023
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sugetha24 sugetha24 sugetha24 approved these changes

Assignees

@carlbuchmann carlbuchmann

Labels
data-model changes module: cv_configlet_v3 Issue related to cv_cononfiglet module in v3 module: cv_configlet Issue related to cv_configlet module module: cv_container_v3 Issue related to cv_container module in v3 module: cv_container Issue related to cv_container module module: cv_device_v3 Issue related to cv_device module in v3 module: cv_device Issue related to cv_device module module: cv_facts_v3 module: cv_facts Issue related to cv_facts module module: cv_task_v3 Issue related to cv_tasks module in v3 module: cv_task Issue related to cv_task module module_utils: cv_client cv_client implementation issue role: configlets_sync Issues related to configlets_sync role role: ztp_configuration Issue related to ztp_configuration role type: documentation Improvements or additions to documentation
Projects
None yet
Milestone
v3.9.0
Development

Successfully merging this pull request may close these issues.
2 participants
@carlbuchmann @sugetha24