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

chore(build): update copyright headers and pre-commit hook #1736 #1740

Merged
merged 1 commit into from
Jan 31, 2025

Conversation

yesudeep
Copy link
Contributor

@yesudeep yesudeep commented Jan 31, 2025

Issue: #1736

The copyright headers in Python files have been updated to include SPDX license identifier and change "Google Inc." to "Google LLC". Additionally, simplified the pre-commit hook command for branch protection.

CHANGELOG:

  • chore: update copyright header to use "Google LLC" instead of "Google Inc."
  • chore: add SPDX-License-Identifier to copyright headers
  • build: simplify no-commits-on-branches pre-commit hook command by removing 'py' argument
  • feat: add check licenses script for later use
  • chore: remove spurious .projectile file that would cause emacs to tighten scope
  • refactor: move PATH environment variable handling into its own function
  • feat: add govulncheck to check for known vulnerabilities in dependencies
  • feat: add cleanup temp files script
  • chore: configure ci checks to run regardless of directory modified to catch cross-runtime errors

Checklist (if applicable):

@yesudeep yesudeep requested review from pavelgj and apascal07 January 31, 2025 20:00
@yesudeep
Copy link
Contributor Author

yesudeep commented Jan 31, 2025

It looks like the picoschema_tests.yaml file has been removed when the dotprompt directory was removed and since the go tests didn't run when that happened, they're failing.

Hmm... It looks like there is common stuff between runtimes and we should run the tests for all runtimes whenever we make changes.

#1695

@yesudeep yesudeep force-pushed the yesudeep/chore/tan-comfort branch 2 times, most recently from abf8f27 to 64dd782 Compare January 31, 2025 20:38
Issue: #1736

The copyright headers in Python files have been updated to include SPDX license
identifier and change "Google Inc." to "Google LLC". Additionally, simplified
the pre-commit hook command for branch protection.

CHANGELOG:
- chore: update copyright header to use "Google LLC" instead of "Google Inc."
- chore: add SPDX-License-Identifier to copyright headers
- build: simplify no-commits-on-branches pre-commit hook command by removing 'py' argument
- feat: add check licenses script for later use
- chore: remove spurious .projectile file that would cause emacs to tighten scope
- refactor: move PATH environment variable handling into its own
  function
- feat: add govulncheck to check for known vulnerabilities in
  dependencies
- feat: add cleanup temp files script
- chore: configure ci checks to run regardless of directory modified to
  catch cross-runtime errors
@yesudeep yesudeep force-pushed the yesudeep/chore/tan-comfort branch from 64dd782 to 7eff9db Compare January 31, 2025 20:39
@yesudeep
Copy link
Contributor Author

Temporarily disabled breaking test and assigned to @hugoaguirre to address.
See: #1741

@yesudeep yesudeep merged commit 03da015 into main Jan 31, 2025
5 checks passed
@yesudeep yesudeep deleted the yesudeep/chore/tan-comfort branch January 31, 2025 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants