Skip to content

Bump yarn to 4.14.1#9996

Merged
Fryguy merged 1 commit into
ManageIQ:masterfrom
asirvadAbrahamVarghese:update-yarn
Apr 21, 2026
Merged

Bump yarn to 4.14.1#9996
Fryguy merged 1 commit into
ManageIQ:masterfrom
asirvadAbrahamVarghese:update-yarn

Conversation

@asirvadAbrahamVarghese
Copy link
Copy Markdown
Contributor

Replaces #9995

@asirvadAbrahamVarghese asirvadAbrahamVarghese requested a review from a team as a code owner April 21, 2026 07:08
Comment thread .yarnrc.yml

enableGlobalCache: true

enableScripts: true
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Setting this to true since the default is false, as mentioned here (related #7089)

@miq-bot
Copy link
Copy Markdown
Member

miq-bot commented Apr 21, 2026

Some comments on commits asirvadAbrahamVarghese/manageiq-ui-classic@629347b~...a248110

.yarn/releases/yarn-4.14.1.cjs

  • ⚠️ - 140 - Detected pp. Remove all debugging statements.
  • ⚠️ - 191 - Detected pp. Remove all debugging statements.
  • ⚠️ - 292 - Detected pp. Remove all debugging statements.
  • ⚠️ - 294 - Detected pp. Remove all debugging statements.
  • ⚠️ - 299 - Detected pp. Remove all debugging statements.

@miq-bot
Copy link
Copy Markdown
Member

miq-bot commented Apr 21, 2026

Checked commits asirvadAbrahamVarghese/manageiq-ui-classic@629347b~...a248110 with ruby 3.3.10, rubocop 1.86.0, haml-lint 0.73.0, and yamllint 1.37.1
1 file checked, 0 offenses detected
Everything looks fine. 👍

@asirvadAbrahamVarghese
Copy link
Copy Markdown
Contributor Author

asirvadAbrahamVarghese commented Apr 21, 2026

I think binaries under .yarn/releases/ should be ignored

Some comments on commits asirvadAbrahamVarghese/manageiq-ui-classic@629347b~...a248110

.yarn/releases/yarn-4.14.1.cjs

  • ⚠️ - 140 - Detected pp. Remove all debugging statements.
  • ⚠️ - 191 - Detected pp. Remove all debugging statements.
  • ⚠️ - 292 - Detected pp. Remove all debugging statements.
  • ⚠️ - 294 - Detected pp. Remove all debugging statements.
  • ⚠️ - 299 - Detected pp. Remove all debugging statements.

@Fryguy
Copy link
Copy Markdown
Member

Fryguy commented Apr 21, 2026

@asirvadAbrahamVarghese LOL I was also working on this - came to the same conclusion as you. Yarn 4.14.0 changed the default for enableScripts. Should we change this to enableScripts in each repo? I'm concerned other things will break.

I notice that while cypress was the visible breakage all of the packages that need to build were not building...i.e. none of these were building without enableScripts: true

➤ YN0000: ┌ Link step
➤ YN0007: │ core-js@npm:3.49.0 must be built because it never has been before or the last one failed
➤ YN0007: │ cypress@npm:15.14.0 must be built because it never has been before or the last one failed
➤ YN0007: │ core-js-pure@npm:3.49.0 must be built because it never has been before or the last one failed
➤ YN0007: │ @parcel/watcher@npm:2.5.6 must be built because it never has been before or the last one failed
➤ YN0007: │ @carbon/feature-flags@npm:1.2.0 must be built because it never has been before or the last one failed
➤ YN0007: │ @carbon/layout@npm:11.51.0 must be built because it never has been before or the last one failed
➤ YN0007: │ @carbon/utilities@npm:0.18.0 must be built because it never has been before or the last one failed
➤ YN0007: │ @carbon/colors@npm:11.50.0 must be built because it never has been before or the last one failed
➤ YN0007: │ core-js@npm:2.6.12 must be built because it never has been before or the last one failed
➤ YN0007: │ @carbon/utils-position@npm:1.3.0 must be built because it never has been before or the last one failed
➤ YN0007: │ @carbon/icon-helpers@npm:10.74.0 must be built because it never has been before or the last one failed
➤ YN0007: │ @carbon/motion@npm:11.44.0 must be built because it never has been before or the last one failed
➤ YN0007: │ @ibm/plex-mono@npm:1.1.0 must be built because it never has been before or the last one failed
➤ YN0007: │ @ibm/plex-sans@npm:1.1.0 must be built because it never has been before or the last one failed
➤ YN0007: │ @ibm/plex-sans-arabic@npm:1.1.0 must be built because it never has been before or the last one failed
➤ YN0007: │ @ibm/plex-sans-devanagari@npm:1.1.0 must be built because it never has been before or the last one failed
➤ YN0007: │ @ibm/plex-sans-hebrew@npm:1.1.0 must be built because it never has been before or the last one failed
➤ YN0007: │ @ibm/plex-sans-thai@npm:1.1.0 must be built because it never has been before or the last one failed
➤ YN0007: │ @ibm/plex-sans-thai-looped@npm:1.1.0 must be built because it never has been before or the last one failed
➤ YN0007: │ @ibm/plex-serif@npm:1.1.0 must be built because it never has been before or the last one failed
➤ YN0007: │ fsevents@patch:fsevents@npm%3A1.2.13#optional!builtin<compat/fsevents>::version=1.2.13&hash=d11327 must be built because it never has been before or the last one failed
➤ YN0007: │ es5-ext@npm:0.10.64 must be built because it never has been before or the last one failed
➤ YN0007: │ @carbon/charts@npm:1.27.3 must be built because it never has been before or the last one failed
➤ YN0007: │ @carbon/icons-react@npm:11.78.0 [5f79f] must be built because it never has been before or the last one failed
➤ YN0007: │ @carbon/grid@npm:11.53.0 must be built because it never has been before or the last one failed
➤ YN0007: │ @carbon/charts-react@npm:1.27.3 [5c616] must be built because it never has been before or the last one failed
➤ YN0007: │ @carbon/type@npm:11.57.0 must be built because it never has been before or the last one failed
➤ YN0007: │ @carbon/themes@npm:11.71.0 must be built because it never has been before or the last one failed
➤ YN0007: │ @carbon/styles@npm:1.104.0 [e02d1] must be built because it never has been before or the last one failed
➤ YN0007: │ @carbon/react@npm:1.105.0 [5c616] must be built because it never has been before or the last one failed
➤ YN0000: └ Completed in 1m 15s

@asirvadAbrahamVarghese
Copy link
Copy Markdown
Contributor Author

Right, like I mentioned with the Node upgrade, repos that actually use Yarn will need updates anyway. But for the ones that don’t pull in deps and are just consumed by manageiq-ui-classic as plugins, I would prefer removing the packageManager field (and probably engines as well)

@Fryguy Fryguy merged commit df6036c into ManageIQ:master Apr 21, 2026
20 of 21 checks passed
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