Skip to content

drop rsvp and fs-extra dependencies #393

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

craigteegarden
Copy link

Remove RSVP dependency using same method as ember-bootstrap/ember-bootstrap#2055

@jelhan jelhan changed the title Drop RSVP and fs-extra dependency drop rsvp and fs-extra dependencies May 29, 2025
Copy link
Collaborator

@jelhan jelhan left a comment

Choose a reason for hiding this comment

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

Looks good to me. But I haven't verified locally that it works as expected. Did you verified locally that it's working?

We don't have any test coverage for the blueprints. We rely on manual testing for those changes.

Chalk version 4.x was updated to 5.x in
ember-bootstrap#325

but Chalk 5 makes breaking changes and can't be used in the same way: https://github.com/chalk/chalk/releases/tag/v5.0.0

Also added a test to run the blueprint, which catches the chalk version issue.
@craigteegarden
Copy link
Author

After running into issues using the blueprint, even with the currently published version, I fixed the issue and added a couple tests for the blueprint itself. I then could run the blueprint and verified it still worked with the blueprint fs changes (also the preprocessor test should now cover that too).

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