Skip to content
This repository has been archived by the owner on Jul 2, 2024. It is now read-only.

DEVPROD-3362 Mark Evergreen ide as deprecated #2276

Merged
merged 9 commits into from
Mar 4, 2024

Conversation

khelif96
Copy link
Contributor

DEVPROD-3362

Description

Marks the Evergreen IDE as deprecated for Ubuntu 1804 workstations and also hides the link for other workstations that don't have the ide installed.

Screenshots

image

@khelif96 khelif96 requested a review from a team February 26, 2024 21:49
Copy link

cypress bot commented Feb 26, 2024

2 flaky tests on run #16036 ↗︎

0 547 10 0 Flakiness 2

Details:

Bump snapshots
Project: Spruce Commit: d526c28f3b
Status: Passed Duration: 20:08 💡
Started: Mar 4, 2024 3:46 PM Ended: Mar 4, 2024 4:06 PM
Flakiness  breadcrumbs.ts • 1 flaky test

View Output Video

Test Artifacts
Viewing a patch > Viewing a user's own patch > Clicking the 'My Patches' breadcrumb goes to the logged in user's Patches Page when the current patch belongs to the logged in user Screenshots Video
Flakiness  distroSettings/host_section.ts • 1 flaky test

View Output Video

Test Artifacts
host section > using legacy ssh > shows an error when selecting an incompatible host communication method Screenshots Video

Review all test suite changes for PR #2276 ↗︎

src/pages/spawn/spawnHost/SpawnHostCard/index.tsx Outdated Show resolved Hide resolved
package.json Outdated
@@ -72,6 +72,7 @@
"@leafygreen-ui/guide-cue": "5.0.5",
"@leafygreen-ui/icon": "11.26.0",
"@leafygreen-ui/icon-button": "15.0.19",
"@leafygreen-ui/info-sprinkle": "^1.0.2",
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this dep be pinned?

Copy link
Contributor

Choose a reason for hiding this comment

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

Personally would prefer not to add another nesting level here and just have src/pages/spawn/spawnHost/SpawnHostCard.tsx and src/pages/spawn/spawnHost/constants.ts. Your call though!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was trying to colocate all of the related files into the same folder but since we're not really consistent about this I undid it.

@khelif96 khelif96 requested a review from sophstad March 4, 2024 15:36
@khelif96 khelif96 merged commit 2dbb4d4 into evergreen-ci:main Mar 4, 2024
5 of 6 checks passed
@khelif96 khelif96 deleted the DEVPROD-3362 branch March 4, 2024 16:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants