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

refactor: [M3-6303] - SRC > Components > ColorPalette #9013

Merged
merged 1 commit into from
Apr 18, 2023

Conversation

cpathipa
Copy link
Contributor

Description 📝

  • Ran the tss-react codemod on ColorPalette.
  • Modernized ColorPalette by using named exports.

How to test 🧪

yarn test
yarn cy:run

@cypress
Copy link

cypress bot commented Apr 17, 2023

2 failed and 4 flaky tests on run #3009 ↗︎

2 147 3 0 Flakiness 4

Details:

refactor: [M3-6303] - SRC > Components > ColorPalette
Project: Cloud Manager E2E Commit: 5710a19f2c
Status: Failed Duration: 14:15 💡
Started: Apr 17, 2023 8:59 PM Ended: Apr 17, 2023 9:13 PM
Failed  delete-volume.spec.ts • 1 failed test

View Output Video

Test Artifacts
volume delete flow > deletes a volume Output Screenshots Video
Failed  update-volume.spec.ts • 1 failed test

View Output Video

Test Artifacts
volume update flow > updates a volume's label and tags Output Screenshots Video
Flakiness  linodes/rescue-linode.spec.ts • 1 flaky test

View Output Video

Test Artifacts
rescue linode > rescue a linode Output Screenshots Video
Flakiness  managed/managed-monitors.spec.ts • 1 flaky test

View Output Video

Test Artifacts
Managed Monitors tab > can delete Managed monitors Output Screenshots Video
Flakiness  volumes/attach-volume.spec.ts • 1 flaky test

View Output Video

Test Artifacts
volume attach and detach flows > attaches a volume to a Linode Output Screenshots Video
Flakiness  volumes/clone-volume.spec.ts • 1 flaky test

View Output Video

Test Artifacts
volume clone flow > clones a volume Output Screenshots Video

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

Copy link
Contributor

@abailly-akamai abailly-akamai left a comment

Choose a reason for hiding this comment

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

👍

@jaalah-akamai jaalah-akamai added the Approved Multiple approvals and ready to merge! label Apr 18, 2023
@cpathipa cpathipa merged commit de0fd7f into linode:develop Apr 18, 2023
@cpathipa cpathipa deleted the M3-6303 branch April 18, 2023 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved Multiple approvals and ready to merge! Ready for Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants