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

Add pre-commit hooks and BSD 3-Clause license #51

Merged
merged 3 commits into from
Dec 14, 2024
Merged

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, and YAML files.

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:

Copy link

@jjaime2 jjaime2 left a comment

Choose a reason for hiding this comment

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

Small inconsistency noted, otherwise looks good.

@amalnanavati amalnanavati merged commit 12c5456 into main Dec 14, 2024
1 check passed
@amalnanavati amalnanavati deleted the amaln/pre-commit branch December 14, 2024 01:13
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