Skip to content

fix HarmonyEvaluatedImportSpecifierDependency#15660

Merged
sokra merged 3 commits into
mainfrom
add-provided-info-tests
Apr 19, 2022
Merged

fix HarmonyEvaluatedImportSpecifierDependency#15660
sokra merged 3 commits into
mainfrom
add-provided-info-tests

Conversation

@vankop

@vankop vankop commented Apr 12, 2022

Copy link
Copy Markdown
Member

What kind of change does this PR introduce?
fix #15633

Did you add tests for your changes?
yes

Does this PR introduce a breaking change?
np

What needs to be documented once your changes are merged?
nothing

@webpack-bot

Copy link
Copy Markdown
Contributor

For maintainers only:

  • This needs to be documented (issue in webpack/webpack.js.org will be filed when merged)
  • This needs to be backported to webpack 4 (issue will be created when merged)

@vankop vankop changed the title add providedInfo test case fix HarmonyEvaluatedImportSpecifierDependency.js Apr 12, 2022
@vankop vankop changed the title fix HarmonyEvaluatedImportSpecifierDependency.js fix HarmonyEvaluatedImportSpecifierDependency Apr 12, 2022
@vankop vankop force-pushed the add-provided-info-tests branch 3 times, most recently from b3ea6bd to 9b9dc06 Compare April 13, 2022 09:51
@vankop vankop force-pushed the add-provided-info-tests branch from 9b9dc06 to 55bebac Compare April 13, 2022 09:55
Comment thread lib/dependencies/HarmonyEvaluatedImportSpecifierDependency.js Outdated
@webpack-bot

Copy link
Copy Markdown
Contributor

@vankop Thanks for your update.

I labeled the Pull Request so reviewers will review it again.

@sokra Please review the new changes.

@sokra

sokra commented Apr 19, 2022

Copy link
Copy Markdown
Member

Thanks

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.

in-operator broken; "property" in object ALWAYS gets inlined as true

3 participants