Skip to content

Fix #504 #506

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

Merged
merged 2 commits into from
Aug 26, 2021
Merged

Fix #504 #506

merged 2 commits into from
Aug 26, 2021

Conversation

japgolly
Copy link
Contributor

Our release GitHub Actions job performs the release for SJS 0.6 and 1.0 in parallel. This prevents each parallel processes from removing each other's deployment.

Our release GitHub Actions job performs the release for SJS 0.6 and 1.0 in parallel. This prevents each parallel processes from removing each other's deployment.
@japgolly japgolly requested a review from armanbilge August 26, 2021 03:11
@japgolly japgolly linked an issue Aug 26, 2021 that may be closed by this pull request
Copy link
Member

@armanbilge armanbilge left a comment

Choose a reason for hiding this comment

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

Never used this before, I trust you know what's what :)

@armanbilge
Copy link
Member

Also, this doesn't completely fix #504 right? We still need to do something about the version (hard-code it I guess)?

@japgolly
Copy link
Contributor Author

Yeah it doesn't fix the version problem but it's only a problem on the 1.x branch and I think it will be less effort to just set the version manually on a release. I'm hoping we don't do another 1.x release anyway :)

@armanbilge
Copy link
Member

Ok I hope so!

@armanbilge armanbilge merged commit 03cb1d6 into series/1.x Aug 26, 2021
@japgolly japgolly deleted the issue/504 branch August 26, 2021 05:52
@@ -17,6 +22,7 @@ lazy val root = project

name := "Scala.js DOM"

ThisBuild / version := "1.2.0"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@armanbilge erm......

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.

(Non-snapshot) release is broken
2 participants