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

Update copyright date for 2025 #151

Merged
merged 1 commit into from
Jan 14, 2025
Merged

Update copyright date for 2025 #151

merged 1 commit into from
Jan 14, 2025

Conversation

jjaime2
Copy link
Contributor

@jjaime2 jjaime2 commented Jan 14, 2025

This PR updates the copyright date on all files managed by our pre-commit workflow.

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


Before creating a pull request

  • Format React code with npm run format
  • Format Python code by running python3 -m black . in the top-level of this repository
  • Thoroughly test your code's functionality, including unintended uses.
  • Fully test the responsiveness of the feature as documented in the Responsiveness Testing Guidelines. If you deviate from those guidelines, document above why you deviated and what you did instead.
  • Consider the user flow between states that this feature introduces, consider different situations that might occur for the user, and ensure that there is no way for the user to get stuck in a loop.

Before merging a pull request

  • Squash all your commits into one (or Squash and Merge)

@jjaime2 jjaime2 requested a review from amalnanavati January 14, 2025 00:57
Copy link
Contributor

@amalnanavati amalnanavati left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@jjaime2 jjaime2 merged commit 0d50ae2 into main Jan 14, 2025
5 checks passed
@jjaime2 jjaime2 deleted the jjaime2/2025-pre-commit branch January 14, 2025 01:06
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