Skip to content
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

Rename stdlib-bootstrapped-tasty to scala2-library-tasty #18615

Merged
merged 2 commits into from
Oct 3, 2023

Conversation

nicolasstucki
Copy link
Contributor

Align with the scala-library and dotty-library nomenclature. Note
that the name of the project/jar is already scala2-library.

  • stdlib-bootstrapped -> scala2-library-bootstrapped
  • stdlib-bootstrapped-tasty -> scala2-library-tasty
  • stdlib-bootstrapped-tasty-tests -> scala2-library-tasty-tests

Based on #18613

This flag only works on the `scala3-bootstrapped/scalac` in the dotty
project. It is intended for internal testing.

Currently it is a bit expensive to compile the Scala 2 library TASTy.
One reason in for this is that we need to use `-Ycheck:all` when
compiling it to make sure that `-Yscala2-stdlib` is working properly.

We only allow `scala3-bootstrapped/scalac` and not `scalac` to have this
dependency for two reasons. First, we do want to be able to compile and
run `scalac` command fast to reduce development testing time. Second, we
do not have a stable enough version of the Scala 2 library TASTy yet.

We also do not enable the Scala 2 library TASTy by default on
`scala3-bootstrapped/scalac` because we still need this command to test
against the current version of the library. Stability is also a concern
for this version of the command.
Align with the `scala-library` and `dotty-library` nomenclature. Note
that the name of the project/jar is already `scala2-library`.

* `stdlib-bootstrapped` -> `scala2-library-bootstrapped`
* `stdlib-bootstrapped-tasty` -> `scala2-library-tasty`
* `stdlib-bootstrapped-tasty-tests` -> `scala2-library-tasty-tests`
@nicolasstucki nicolasstucki marked this pull request as ready for review September 29, 2023 13:09
@odersky odersky merged commit 48bb59c into scala:main Oct 3, 2023
36 checks passed
@odersky odersky deleted the rename-stdlib-bootstrapped branch October 3, 2023 20:01
nicolasstucki added a commit to dotty-staging/dotty that referenced this pull request Oct 17, 2023
Follow up name changes in scala#18615 and align with flag added in scala#18613.
bishabosha added a commit that referenced this pull request Oct 23, 2023
Follow up name changes in #18615 and align with flag added in #18613.
@Kordyjan Kordyjan added this to the 3.4.0 milestone Dec 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants