Skip to content

Handle imports that don’t begin with a “~” correctly#596

Merged
christian-bromann merged 1 commit intostenciljs:mainfrom
alexandertrefz:fix/custom-importer
Mar 7, 2025
Merged

Handle imports that don’t begin with a “~” correctly#596
christian-bromann merged 1 commit intostenciljs:mainfrom
alexandertrefz:fix/custom-importer

Conversation

@alexandertrefz
Copy link
Copy Markdown
Contributor

@alexandertrefz alexandertrefz commented Mar 6, 2025

Pull request checklist

Please check if your PR fulfills the following requirements:

  • Build (npm run build) was run locally and any changes were pushed
  • Tests (npm test) were run locally and passed
  • Prettier (npm run prettier) was run locally and passed

Pull request type

Please check the type of change your PR introduces:

  • Bugfix

What is the current behavior?

@forward or @use-ing 2 files from the same file creates a Type Error.

What is the new behavior?

The file compiles correctly.

Does this introduce a breaking change?

  • No

@alexandertrefz alexandertrefz requested a review from a team as a code owner March 6, 2025 23:29
@alexandertrefz alexandertrefz mentioned this pull request Mar 6, 2025
5 tasks
Copy link
Copy Markdown
Member

@christian-bromann christian-bromann left a comment

Choose a reason for hiding this comment

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

Awesome 👍

@christian-bromann christian-bromann merged commit e357bc1 into stenciljs:main Mar 7, 2025
4 checks passed
@christian-bromann
Copy link
Copy Markdown
Member

I think it would make sense to get some tests in place here but happy to have this as a separate PR.

@alexandertrefz alexandertrefz deleted the fix/custom-importer branch March 12, 2025 00:55
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