Skip to content

Conversation

lizlooney
Copy link
Collaborator

This pr breaks up common_storage.ts into:

  • names.ts
  • project.ts
  • module.ts
  • module_content.ts

There is a separate commit for each of the files, so (if you want) you can review each commit.

@lizlooney lizlooney requested a review from alan412 August 10, 2025 17:25
Copy link
Collaborator

@alan412 alan412 left a comment

Choose a reason for hiding this comment

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

LGTM. The only changes needed are the copyright year in the license statements but I approved so you can just make those changes and merge.

@lizlooney lizlooney merged commit 02662fe into wpilibsuite:main Aug 11, 2025
1 check passed
@lizlooney lizlooney deleted the pr_breakup_common_storage branch August 11, 2025 00:38
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