-
Notifications
You must be signed in to change notification settings - Fork 246
feat(web, ci): add publish script; automatically publish for dev on every merge to main COMPASS-10156 #7637
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
Conversation
1c9d87b to
44fa34b
Compare
…e to its own file; adjust tsconfig
2503463 to
23b1123
Compare
23b1123 to
51a46ac
Compare
|
Before disabling the publish for patches I manually tested that it works: https://downloads.mongodb.com/compass/compass-web/dev/index.mjs Also this PR in mms uses this URL successfully https://github.com/10gen/mms/pull/150184 |
| E2E_TESTS_ATLAS_CS_WITHOUT_SEARCH: ${e2e_tests_atlas_cs_without_search} | ||
| E2E_TESTS_ATLAS_CS_WITH_SEARCH: ${e2e_tests_atlas_cs_with_search} | ||
|
|
||
| parameters: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using parameters for these values so that later down the road we will be able to use those when spawning patches for releases with evergreen cli
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Logic here is the same as before, but I moved some things around and git thinks it's a completely new file now 😓
ivandevp
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This patch adds a new publish script for compass-web that will upload the special compass-web entrypoint js file that points to one of the previously uploaded releases