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

Allow ember-data type registry imports in use-ember-data-rfc-395-imports rule #2027

Merged
merged 2 commits into from
Dec 12, 2023

Conversation

wagenet
Copy link
Contributor

@wagenet wagenet commented Dec 11, 2023

Addresses the issue raised by #1833, though we require the import to be directly from the registry module.

@bmish
Copy link
Member

bmish commented Dec 11, 2023

Can you target the v11 branch instead of master? Then I can release this as a bug fix to v11, as v12 on master is still WIP.

@bmish bmish changed the base branch from master to v11 December 12, 2023 22:54
@bmish bmish changed the base branch from v11 to master December 12, 2023 22:54
@bmish bmish changed the base branch from master to v11 December 12, 2023 23:00
@bmish bmish changed the title Allow ember-data type registry imports Allow ember-data type registry imports in use-ember-data-rfc-395-imports rule Dec 12, 2023
@bmish bmish merged commit a1f7f12 into ember-cli:v11 Dec 12, 2023
@bmish bmish mentioned this pull request Dec 12, 2023
bmish added a commit that referenced this pull request Dec 12, 2023
* Support ESLint flat config (#2020)

* Allow ember-data type registry imports in `use-ember-data-rfc-395-imports` rule (#2027)

* Allow ember-data type registry imports

* fix lint

---------

Co-authored-by: Bryan Mishkin <698306+bmish@users.noreply.github.com>

* fix lint

* Release 11.12.0

* fix merge mistakes

---------

Co-authored-by: Peter Wagenet <peter.wagenet@gmail.com>
@wagenet wagenet deleted the ember-data-type-registries branch January 4, 2024 17:00
@wagenet
Copy link
Contributor Author

wagenet commented Jan 4, 2024

@bmish whoops, sorry I missed this. Thanks for taking care of it!

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.

2 participants