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

Upgrade to yarn 3 #7904

Merged
merged 3 commits into from
Oct 19, 2021
Merged

Upgrade to yarn 3 #7904

merged 3 commits into from
Oct 19, 2021

Conversation

Fryguy
Copy link
Member

@Fryguy Fryguy commented Oct 8, 2021

Part of #7903

@miq-bot
Copy link
Member

miq-bot commented Oct 8, 2021

Some comments on commits Fryguy/manageiq-ui-classic@d66a46d~...d3e1399

.yarn/releases/yarn-3.0.2.cjs

  • ⚠️ - 311 - Detected pp. Remove all debugging statements.
  • ⚠️ - 312 - Detected pp. Remove all debugging statements.
  • ⚠️ - 316 - Detected pp. Remove all debugging statements.
  • ⚠️ - 382 - Detected pp. Remove all debugging statements.
  • ⚠️ - 386 - Detected pp. Remove all debugging statements.
  • ⚠️ - 435 - Detected pp. Remove all debugging statements.
  • ⚠️ - 58 - Detected pp. Remove all debugging statements.
  • ⚠️ - 99 - Detected pp. Remove all debugging statements.

lib/tasks/manageiq/ui_tasks.rake

  • ⚠️ - 9 - Detected puts. Remove all debugging statements.

@miq-bot
Copy link
Member

miq-bot commented Oct 8, 2021

Checked commits Fryguy/manageiq-ui-classic@d66a46d~...d3e1399 with ruby 2.6.3, rubocop 1.13.0, haml-lint 0.35.0, and yamllint
1 file checked, 0 offenses detected
Everything looks fine. ⭐

miq-bot pushed a commit to ManageIQ/manageiq-cross_repo-tests that referenced this pull request Oct 8, 2021
miq-bot pushed a commit to ManageIQ/manageiq-cross_repo-tests that referenced this pull request Oct 8, 2021
@Fryguy
Copy link
Member Author

Fryguy commented Oct 8, 2021

Cross repo tests are green - I still want to do some more manual tests and verify certain things like direct yarn and npm commands still working

@kavyanekkalapu
Copy link
Member

kavyanekkalapu commented Oct 19, 2021

@Fryguy Looks good to me. I loaded all the prs (ui-classic + providers + v2v) in my local and did some random testing in ui pages. It is working as expected.

@kavyanekkalapu kavyanekkalapu merged commit e4a9574 into ManageIQ:master Oct 19, 2021
@Fryguy Fryguy deleted the yarn3 branch October 22, 2021 12:58
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