Skip to content

Conversation

@clutroth
Copy link
Contributor

No description provided.

@clutroth clutroth force-pushed the scalajs-1.7.0 branch 2 times, most recently from 587e085 to a49232b Compare October 20, 2021 13:55
@clutroth clutroth marked this pull request as ready for review October 20, 2021 14:11
Copy link
Contributor

@gzm0 gzm0 left a comment

Choose a reason for hiding this comment

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

Only minor stuff.

scalaversion: ["2.11.12", "2.12.10", "2.13.1"]
scalajsversion: ["1.5.0"]
scalaversion: ["2.11.12", "2.12.15", "2.13.6"]
scalajsversion: ["1.7.0", "1.7.1"]
Copy link
Contributor

Choose a reason for hiding this comment

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

I do not quite understand how this is supposed to work: Won't we publish the CLI for a single Scala.js version only anyways? If so, what's the point of testing with an older one?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You're right. It's pointless.

@@ -1 +1 @@
sbt.version=1.2.8
sbt.version=1.5.5
Copy link
Contributor

Choose a reason for hiding this comment

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

Put the sbt version upgrade into a separate commit to reduce noise?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@gzm0
Copy link
Contributor

gzm0 commented Oct 22, 2021

@clutroth thank you for your contributions to the CLI. I realize we have not even tagged a release after your last PR. Apologies for that.

@sjrd, I suggest we simply start releasing on its own versioning scheme? maybe with minor version bumps if we upgrade the minor Scala.js version? Definitely not the cleanest option, but better than not releasing at all :-/ WDYT?

@sjrd
Copy link
Member

sjrd commented Oct 22, 2021

Yes, probably that's what we should do.

Ideally I think we should explore something using cs install so that we only have to release to Maven Central, and not deal with the native runner scripts.

@clutroth
Copy link
Contributor Author

clutroth commented Nov 2, 2021

Glad to help :) I'd appreciate release.

@clutroth clutroth requested a review from gzm0 November 2, 2021 12:25
@gzm0 gzm0 merged commit 9c1a30a into scala-js:master Nov 2, 2021
@gzm0 gzm0 mentioned this pull request Nov 2, 2021
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.

3 participants