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

Angular: Retrieve version from core package #17363

Merged
merged 1 commit into from
Jan 29, 2022

Conversation

matheo
Copy link
Member

@matheo matheo commented Jan 28, 2022

Issue:

What I did

I have a nx v13.1.3 monorepo and today I've tried to update @storybook/* from ~6.3.0 to ~6.4.0 and got an error:
image

Solution

Looking at the mentioned file (node_modules/@storybook/angular/dist/ts3.9/server/framework-preset-angular-cli.js)
I see that it tries to extract the VERSION of the @angular/cli package but my Nx monorepo doesn't have it.
I also see that it's just an indication to use different files of @angular-devkit/build-angular but that package doesn't expose its version, so I got a successful build just extracting the version from @angular/core instead, which should be similar to the CLI in use :)

@nx-cloud
Copy link

nx-cloud bot commented Jan 28, 2022

☁️ Nx Cloud Report

CI ran the following commands for commit 12e4b28. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch


✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

@shilman shilman changed the title Angular: retrieve version from the core package Angular: etrieve version from the core package Jan 29, 2022
@shilman shilman changed the title Angular: etrieve version from the core package Angular: Retrieve version from the core package Jan 29, 2022
@shilman shilman changed the title Angular: Retrieve version from the core package Angular: Retrieve version from core package Jan 29, 2022
@shilman shilman added angular bug nx Prioritize Nx compatibility for angular patch:yes Bugfix & documentation PR that need to be picked to main branch labels Jan 29, 2022
Copy link
Member

@shilman shilman left a comment

Choose a reason for hiding this comment

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

LGTM. Nice fix @matheo !!! 🙏

@shilman shilman merged commit b4a70f0 into storybookjs:next Jan 29, 2022
@shilman shilman added the patch:done Patch/release PRs already cherry-picked to main/release branch label Jan 29, 2022
shilman added a commit that referenced this pull request Jan 29, 2022
Angular: Retrieve version from core package
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
angular bug nx Prioritize Nx compatibility for angular patch:done Patch/release PRs already cherry-picked to main/release branch patch:yes Bugfix & documentation PR that need to be picked to main branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants