Skip to content

Comments

Added pre-commit and license#150

Merged
amalnanavati merged 4 commits intomainfrom
amaln/pre-commit
Dec 14, 2024
Merged

Added pre-commit and license#150
amalnanavati merged 4 commits intomainfrom
amaln/pre-commit

Conversation

@amalnanavati
Copy link
Contributor

@amalnanavati amalnanavati commented Dec 12, 2024

This PR adds pre-commit hooks for consistent formatting. It also adds a BSD 3-Clause License to this repository, and a license header on top of all Python, XML, YAML, JS(X), HTML, and CSS files. It also upgrades the versions of some packages to work better with pre-commit.

See ada_feeding#206 for testing. Specifically, before merging any of these pre-commit/license PRs in, we ensured the entire feeding demo still works in both real and mock.

Related PRs:

"version": "4.21.5",
"resolved": "https://registry.npmjs.org/browserslist/-/browserslist-4.21.5.tgz",
"integrity": "sha512-tUkiguQGW7S3IhB7N+c2MV/HZPSCPAAiYBZXLsBhFB/PCy6ZKKsZrmBayHV9fdGV/ARIfJ14NkxKzRDjvp7L6w==",
"version": "4.24.2",
Copy link
Contributor

Choose a reason for hiding this comment

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

Are these automated version updates?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I did them. Pre-commit was initially failing due to a versioning issue, which this addressed.

@amalnanavati amalnanavati merged commit 28f9fe6 into main Dec 14, 2024
@amalnanavati amalnanavati deleted the amaln/pre-commit branch December 14, 2024 02:05
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