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

Updating copyright across all files based on when it was first introduced. #2027

Merged
merged 4 commits into from
Mar 13, 2024

Conversation

Atlas-64
Copy link
Contributor

@Atlas-64 Atlas-64 commented Mar 11, 2024

Updates copyright across all source files to list the year the file was introduced in adherence to Google's policy. Additionally, changes regex to address any arising linter violations.

closes issue: #1926

Checklist

  • No CHANGELOG update needed
  • No new tests needed
  • No documentation update needed

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Please add bug fixes, new features, breaking changes and anything else you think is worthwhile mentioning to the master (unreleased) section of CHANGELOG.md. If no CHANGELOG update is needed add the following to the PR description: [x] No CHANGELOG update needed

@github-actions github-actions bot dismissed their stale review March 11, 2024 19:15

CHANGELOG updated or no update needed, thanks! 😄

@Atlas-64
Copy link
Contributor Author

@mr-tz it's failing the linting check i wonder what could be causing it to return the copyright is not present , maybe the regex in the flake8.ini not updated to handle the the linter violations ?

@mr-tz
Copy link
Collaborator

mr-tz commented Mar 12, 2024

yes, see the original issue post

@Atlas-64
Copy link
Contributor Author

yes, see the original issue post

okay cool will make changes in the flake8.ini file to reflect the changes in copyright across all the files

@Atlas-64 Atlas-64 changed the title Updating copyright of files based on when it was first introduced. Updating copyright across all files based on when it was first introduced. Mar 12, 2024
Copy link
Collaborator

@mr-tz mr-tz 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!!

@mr-tz mr-tz merged commit 0eb4291 into mandiant:master Mar 13, 2024
28 checks passed
@mr-tz mr-tz mentioned this pull request Mar 13, 2024
@mr-tz mr-tz mentioned this pull request Mar 22, 2024
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants