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

docs(config): update updating guide for transformAliasedImportPaths #1128

Merged
merged 2 commits into from
Jun 26, 2023

Conversation

rwaskiewicz
Copy link
Contributor

@rwaskiewicz rwaskiewicz commented May 24, 2023

this commit updates the upgrade guide for stencil v4 to include new default values for transformAliasedImportPaths and transformAliasedImportPathsInCollection.

STENCIL-829

Core PR: stenciljs/core#4418

@vercel
Copy link

vercel bot commented May 24, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
stencil-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 26, 2023 3:31pm

rwaskiewicz added a commit to stenciljs/core that referenced this pull request May 24, 2023
@rwaskiewicz rwaskiewicz force-pushed the rwaskiewicz/v4-dev-init branch from f9ff307 to 97d0c67 Compare May 30, 2023 15:30
Base automatically changed from rwaskiewicz/v4-dev-init to main May 30, 2023 15:34
rwaskiewicz added a commit to stenciljs/core that referenced this pull request Jun 2, 2023
rwaskiewicz added a commit to stenciljs/core that referenced this pull request Jun 2, 2023
rwaskiewicz added a commit to stenciljs/core that referenced this pull request Jun 21, 2023
rwaskiewicz added a commit to stenciljs/core that referenced this pull request Jun 21, 2023
@rwaskiewicz rwaskiewicz force-pushed the rwaskiewicz/transformAliasedPathsTrueDocs branch from 3c7002e to 8e949d1 Compare June 22, 2023 13:09
@rwaskiewicz rwaskiewicz marked this pull request as ready for review June 22, 2023 13:09
@rwaskiewicz rwaskiewicz requested a review from a team as a code owner June 22, 2023 13:09
@rwaskiewicz rwaskiewicz requested review from alicewriteswrongs and tanner-reits and removed request for a team June 22, 2023 13:09
Copy link
Contributor

@tanner-reits tanner-reits left a comment

Choose a reason for hiding this comment

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

What are your thoughts on removing the flags specifically for dist and dist-collection and having all of the transform behavior driven by the top-level config flag? I think it's confusing having them at different levels

:::tip
If using the `dist-collection` output target directly, the same result can be achieved using the `transformAliasedImportPaths` flag on the target's config.
If using the `dist-collection` output target directly, the same result can be achieved using the [`transformAliasedImportPaths`](../config/01-overview.md#transformaliasedimportpaths) flag on the target's config.
Copy link
Contributor

Choose a reason for hiding this comment

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

This link points to the top-level config flag not the one for the dist-collection target specifically.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍 4090dda

@rwaskiewicz
Copy link
Contributor Author

What are your thoughts on removing the flags specifically for dist and dist-collection and having all of the transform behavior driven by the top-level config flag? I think it's confusing having them at different levels

I think it's probably the right move, but I think we're a little too close to the release to change this right now. I think enabling them by default will mitigate this a little bit (it's just "on"), and something I'd be interested in coalesing for v5

@rwaskiewicz rwaskiewicz requested a review from tanner-reits June 22, 2023 20:54
this commit updates the upgrade guide for stencil v4 to include new
default values for `transformAliasedImportPaths` and
`transformAliasedImportPathsInCollection`.

STENCIL-829
@rwaskiewicz rwaskiewicz force-pushed the rwaskiewicz/transformAliasedPathsTrueDocs branch from 4090dda to 1cfaa11 Compare June 26, 2023 15:29
@rwaskiewicz rwaskiewicz merged commit c872f4f into main Jun 26, 2023
@rwaskiewicz rwaskiewicz deleted the rwaskiewicz/transformAliasedPathsTrueDocs branch June 26, 2023 15:34
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.

3 participants