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

Disable no-curly-component-invocation and no-implicit-this rules for gjs / gts files #2844

Merged
merged 4 commits into from Mar 15, 2023

Conversation

NullVoxPopuli
Copy link
Contributor

@NullVoxPopuli NullVoxPopuli commented Mar 14, 2023

Fixes #2830

Normally this would be a breaking change under any other circumstances, but gjs/gts folks need to disable these rules anyway, so this is more a bugfix.

…template-lint#2830

Normally this would be a breaking change under any other circumstances, but gjs/gts folks need to disable these rules anyway, so this is more a bugfix.
@bmish bmish changed the title Start adding gjs/gts overrides to the recommended config, per: #2830 Disable no-curly-component-invocation and no-implicit-this rules for gjs / gts files Mar 15, 2023
@bmish bmish added the bug label Mar 15, 2023
Comment on lines +98 to +99
'no-curly-component-invocation': 'off',
'no-implicit-this': 'off',
Copy link
Member

Choose a reason for hiding this comment

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

Mind adding a comment next to each of these rules explaining why they are disabled for gjs/gts?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated!

Copy link
Member

@bmish bmish Mar 15, 2023

Choose a reason for hiding this comment

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

Thanks. Lint failing.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

that's what I get for trying to do this all in-browser 🙃

Copy link
Member

@bmish bmish left a comment

Choose a reason for hiding this comment

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

Thanks!

@bmish bmish changed the title Disable no-curly-component-invocation and no-implicit-this rules for gjs / gts files Disable no-curly-component-invocation and no-implicit-this rules for gjs / gts files (#2844)Co-authored-by: Bryan Mishkin <698306+bmish@users.noreply.github.com> Mar 15, 2023
@bmish bmish merged commit f49d3e5 into ember-template-lint:master Mar 15, 2023
@bmish bmish changed the title Disable no-curly-component-invocation and no-implicit-this rules for gjs / gts files (#2844)Co-authored-by: Bryan Mishkin <698306+bmish@users.noreply.github.com> Disable no-curly-component-invocation and no-implicit-this rules for gjs / gts files Mar 15, 2023
@bmish bmish added bug and removed bug labels Mar 15, 2023
@ynotdraw
Copy link

For some strange reason I'm still having to override this in an app using gts/gjs https://github.com/CrowdStrike/ember-toucan-core/pull/29/files#diff-47149a39ab889db56d9810a099412dea44af3fa4b519ac990f8841c10c000ad6. Am I missing something?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Should we start a Polaris config that folks can opt in to when using gjs/gts?
3 participants