Skip to content

CI: Build latest docker image rather than tot#1359

Merged
sbc100 merged 1 commit into
mainfrom
docker_latest
Mar 14, 2024
Merged

CI: Build latest docker image rather than tot#1359
sbc100 merged 1 commit into
mainfrom
docker_latest

Conversation

@sbc100
Copy link
Copy Markdown
Collaborator

@sbc100 sbc100 commented Mar 14, 2024

This means that CI run that update latest actually test the thing we are about to ship.

We recently had a case where latest was broken but tot was fixes and we accidentally shipped a broken SDK version (#1353).

@sbc100 sbc100 requested review from aheejin and kripken March 14, 2024 16:11
@sbc100
Copy link
Copy Markdown
Collaborator Author

sbc100 commented Mar 14, 2024

I expect the CI for this to fail because latest is currently broken, however I think we should land this anyway so that when we update latest we can see that it gets fixed.

This means that CI run that update `latest` actually test the thing
we are about to ship.

We recently had a case where `latest` was broken but `tot` was fixes
and we accidentally shipped a broken SDK version (#1353).
@sbc100 sbc100 merged commit 44bf7cf into main Mar 14, 2024
@sbc100 sbc100 deleted the docker_latest branch March 14, 2024 16:45
mmorel-35 pushed a commit to mmorel-35/emsdk that referenced this pull request Feb 3, 2026
This means that CI run that update `latest` actually test the thing
we are about to ship.

We recently had a case where `latest` was broken but `tot` was fixes
and we accidentally shipped a broken SDK version (emscripten-core#1353).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants