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

Implement support for embedded templates #2483

Merged

Conversation

ventuno
Copy link
Contributor

@ventuno ventuno commented Apr 22, 2022

Continue the work started in #2324

@ventuno ventuno force-pushed the implement-support-embedded-templates branch from 93394ea to 17e1946 Compare April 24, 2022 22:52
@ventuno ventuno force-pushed the implement-support-embedded-templates branch from 9069f66 to 4fa83d0 Compare April 24, 2022 23:20
Copy link
Contributor

@NullVoxPopuli NullVoxPopuli left a comment

Choose a reason for hiding this comment

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

Looking good! just a couple questions

lib/linter.js Outdated Show resolved Hide resolved
test/acceptance/editors-integration-test.js Outdated Show resolved Hide resolved
test/acceptance/editors-integration-test.js Show resolved Hide resolved
lib/extract-templates.js Outdated Show resolved Hide resolved
lib/linter.js Outdated Show resolved Hide resolved
lib/linter.js Outdated Show resolved Hide resolved
lib/extract-templates.js Show resolved Hide resolved
lib/extract-templates.js Show resolved Hide resolved
lib/extract-templates.js Show resolved Hide resolved
lib/extract-templates.js Show resolved Hide resolved
lib/linter.js Outdated Show resolved Hide resolved
lib/linter.js Outdated Show resolved Hide resolved
lib/rules/_base.js Show resolved Hide resolved
test/acceptance/editors-integration-test.js Show resolved Hide resolved
@ventuno ventuno requested review from rwjblue, chriskrycho and NullVoxPopuli and removed request for chriskrycho and NullVoxPopuli August 13, 2022 15:48
@ventuno ventuno requested review from rwjblue and chriskrycho and removed request for chriskrycho and rwjblue August 29, 2022 20:08
@rwjblue rwjblue merged commit 2ba29bd into ember-template-lint:master Aug 31, 2022
@ventuno ventuno deleted the implement-support-embedded-templates branch August 31, 2022 16:10
@bmish bmish changed the title Implement support embedded templates Implement support for embedded templates Aug 31, 2022
rwjblue added a commit to ember-template-imports/ember-template-imports that referenced this pull request Oct 26, 2022
ember-template-lint@4.14.0 landed a direct import of this file as `import * as util from 'ember-template-imports/src/util.js';` (in ember-template-lint/ember-template-lint#2483), but #75 removed `src/utils.js` (migrating it to TypeScript).

In order to prevent users from getting errors when getting a floating version of `ember-template-imports` that includes this file being rewritting in `.ts` we need to re-export from `lib/utils.js`.

This should be removed in the next major version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants