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

Fix issue while saving gjs/gts where prettier deletes the contents of the file: Downgrade prettier-plugin-ember-template-tag until we're on prettier@v3 #185

Merged
merged 2 commits into from
Aug 18, 2023

Conversation

NullVoxPopuli
Copy link
Collaborator

@NullVoxPopuli NullVoxPopuli commented Aug 17, 2023

Temporary alternative to:

Still resolves:

Note that our tests currently don't catch this class of problem.
Only running lint:fix would catch this, and ember-cli doesn't run lint:fix after generating an addon: ember-cli/ember-cli#10334 (I need to come back to this)


it('build and test ', async () => {
// Copy over fixtures
await helper.fixtures.use('./my-addon/src/components');
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

re-copying over the fixtures here hid this problem from us in #159

Copy link
Contributor

@void-mAlex void-mAlex left a comment

Choose a reason for hiding this comment

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

lgtm

@NullVoxPopuli NullVoxPopuli merged commit 1a9dcd9 into main Aug 18, 2023
15 checks passed
@NullVoxPopuli NullVoxPopuli deleted the downgrade-prettier-plugin-template-tag branch August 18, 2023 14:12
@NullVoxPopuli NullVoxPopuli added the bug Something isn't working label Aug 18, 2023
@NullVoxPopuli NullVoxPopuli changed the title Until we're on prettier v3, we can't use prettier-plugin-ember-template-tag@v1 Fix issue while saving gjs/gts where prettier deletes the contents of the file: Downgrade prettier-plugin-ember-template-tag until we're on prettier@v3 Aug 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants