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(npm): expost package_json.bzl and resolved.json for each package alias #1857

Merged
merged 1 commit into from
Jul 24, 2024

Conversation

jbedard
Copy link
Member

@jbedard jbedard commented Jul 20, 2024

Different aliases of a package, potentially being different versions, should each have their own @npm//:pkg-alias/resolved.json and @npm//:pkg-alias/package_json.bzl.

Previously each alias of a package would write to the same files of the real package name causing a last-write-win and aliases not having these two files.


Changes are visible to end-users: no

Test plan

  • New test cases added

@jbedard jbedard requested a review from gregmagolan July 20, 2024 05:45
@jbedard jbedard force-pushed the resolved-aliases branch from 2342cfb to e99aac3 Compare July 20, 2024 05:49
@jbedard jbedard marked this pull request as draft July 20, 2024 05:50
@jbedard jbedard requested a review from thesayyn July 20, 2024 05:50
@jbedard jbedard force-pushed the resolved-aliases branch from e99aac3 to ddd8bd8 Compare July 20, 2024 05:56
@jbedard jbedard marked this pull request as ready for review July 20, 2024 05:56
@jbedard jbedard force-pushed the resolved-aliases branch 2 times, most recently from 689a604 to 3d627d9 Compare July 20, 2024 06:53
@jbedard jbedard changed the title fix(npm): expose resolved.json for each alias of a package fix(npm): expost package_json.bzl and resolved.json for each package alias Jul 20, 2024
@jbedard jbedard force-pushed the resolved-aliases branch from 6748434 to 34ca7cb Compare July 24, 2024 16:33
Copy link
Member

@gregmagolan gregmagolan left a comment

Choose a reason for hiding this comment

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

🌮

@jbedard jbedard merged commit 386e1ec into aspect-build:main Jul 24, 2024
98 checks passed
@jbedard jbedard deleted the resolved-aliases branch July 24, 2024 22:10
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