-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Mitration from nx 17.3.2 to 18.3.5 breaks compilation (input production env NX_MF_DEV_SERVER_STATIC_REMOTES) #27899
Comments
This seems like it may be a caching issue or a stale
is invalidating the stale graph and allowing for a rebuild, as The other thing to check is whether "production" is in your |
Thank you for your response. After reporting the issue, I continued testing and investigating. It seems that the error is caused because there is no configuration for "namedInputs". Shouldn't this be added automatically during the migration? If I remove these two lines, everything works perfectly: And if we don't want to remove those two lines, we need to configure the namedInputs. I took these from a new project generated with nx@19. Shouldn't this be added automatically during the migration?: Once I fixed that, I encountered a problem related to routing. The static servers for the applications that I haven't specified in "devRemotes" are not loading correctly. I've been reviewing the configuration of all the nx files, but I can't find the issue. I've seen related posts, but I can't seem to solve it... With a freshly generated project, routing works correctly, but after migrating the old project, it doesn't. When I try to access any application, the same one always loads, regardless of which one I try to access. However, if I access an application specified in --devRemotes, that one works correctly... Any ideas? Thanks |
There is an issue with the migration but it is not that it needs to add those namedInputs, rather it should only add them if they exist. Can you open a new issue for the remotes issue, with the version of Nx, and as much information as possible, including errors in the terminal and browser |
<!-- Please make sure you have read the submission guidelines before posting an PR --> <!-- https://github.com/nrwl/nx/blob/master/CONTRIBUTING.md#-submitting-a-pr --> <!-- Please make sure that your commit message follows our format --> <!-- Example: `fix(nx): must begin with lowercase` --> <!-- If this is a particularly complex change or feature addition, you can request a dedicated Nx release for this pull request branch. Mention someone from the Nx team or the `@nrwl/nx-pipelines-reviewers` and they will confirm if the PR warrants its own release for testing purposes, and generate it for you if appropriate. --> ## Current Behavior <!-- This is the behavior we have today --> Should use correct namedInputs in the migration instead of assuming 'production' is available. ## Expected Behavior <!-- This is the behavior we should expect with the changes in this PR --> Use the correct namedInputs based on what is listed in `nx.json` ## Related Issue(s) <!-- Please link the issue being fixed so it gets closed when this is merged. --> Fixes #27899 (cherry picked from commit 0a8202a)
This issue has been closed for more than 30 days. If this issue is still occuring, please open a new issue with more recent context. |
Current Behavior
The compilation broke after the update using "nx migrate --run-migrations".
Expected Behavior
It should compile.
GitHub Repo
No response
Steps to Reproduce
I'm not sure if the error actually originates from version 17, as I am migrating step by step from version 15. It's a monorepository with Angular applications. I believe I have pinpointed the problem, but I'm not sure if I need to configure something else or if I should simply remove the configuration that is added in a specific step of the migration.
In any case, it's possible that this migration step is causing the same issue for other users.
Nx Report
Failure Logs
Package Manager Version
No response
Operating System
Additional Information
First of all, I apologize for my level of English; it’s not my native language.
I'm not sure if the error actually originates from version 17, as I am migrating step by step from version 15. It's a monorepository with Angular applications. I believe I have pinpointed the problem, but I'm not sure if I need to configure something else or if I should simply remove the configuration that is added in a specific step of the migration.
If I remove that configuration from the nx.json file, everything works correctly.
I think that configuration is added in one of these two migrations after running the command "nx migrate --run-migrations":
[...]
[...]
[...]
I think it’s related to this: #21390
Thank you
The text was updated successfully, but these errors were encountered: