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.

Sign up for GitHub

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

Jump to bottom

Simplified test fixtures #66

Merged
merged 5 commits into from
Oct 11, 2024
Merged

Simplified test fixtures #66

merged 5 commits into from
Oct 11, 2024

Conversation

ijlee2
Copy link
Owner

@ijlee2 ijlee2 commented Oct 11, 2024 •
edited
Loading

What changed?

  • Provided a realistic example of a v1 addon
  • Provided a realistic example of an engine
  • Removed unnecessary test fixtures for apps, v1 addons, and engines
  • Added unit tests for createFilePathMaps() and updatePathsInAppFolder() (in the case of v1 addon)

@ijlee2 ijlee2 added the enhance: documentation Issue asks for better documentation (e.g. README, code, tests) label Oct 11, 2024
@ijlee2 ijlee2 marked this pull request as ready for review October 11, 2024 17:53
@ijlee2 ijlee2 merged commit 6a9c5e7 into main Oct 11, 2024
2 checks passed
@ijlee2 ijlee2 deleted the simplify-codemod-part-4 branch October 11, 2024 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhance: documentation Issue asks for better documentation (e.g. README, code, tests)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ijlee2