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(no-duplicates): merge type import as with value import #204

Merged
merged 4 commits into from
Dec 17, 2024

Conversation

privatenumber
Copy link

@privatenumber privatenumber commented Dec 12, 2024

There was a bug when merging a type-import with a value-import, when the type import uses import-as syntax.

This handles the import-as case.

Copy link

changeset-bot bot commented Dec 12, 2024

🦋 Changeset detected

Latest commit: f17bfdb

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
eslint-plugin-import-x Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

codesandbox-ci bot commented Dec 12, 2024

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

@privatenumber privatenumber changed the title failing test: no-duplicates auto-fix bug fix(no-duplicates): merge type import as with value import Dec 17, 2024
Copy link
Collaborator

@SukkaW SukkaW left a comment

Choose a reason for hiding this comment

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

LGTM! Would you like to include a changeset to describe your changes? This is for automated changelog generation.

You can add a changeset through yarn changeset.

@SukkaW SukkaW merged commit 32e9b55 into un-ts:master Dec 17, 2024
21 checks passed
@privatenumber
Copy link
Author

Was the CI supposed to release it? Looks like it didn't

@SukkaW
Copy link
Collaborator

SukkaW commented Dec 17, 2024

Was the CI supposed to release it? Looks like it didn't

The changeset action will create a release PR (the current one is #202). I will merge that to make a new release!

@privatenumber
Copy link
Author

Ah I see. I appreciate it, thank you!

renovate bot added a commit to mmkal/eslint-plugin-mmkal that referenced this pull request Dec 17, 2024
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