Skip to content

Add the test suite for scala3-compiler-nonbootstrapped and scala3-compiler-bootstrapped #23916

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 23 commits into from
Sep 15, 2025
Merged

Add the test suite for scala3-compiler-nonbootstrapped and scala3-compiler-bootstrapped #23916

merged 23 commits into from
Sep 15, 2025

Conversation

hamzaremmal
Copy link
Member

No description provided.

@hamzaremmal hamzaremmal self-assigned this Sep 13, 2025
@hamzaremmal hamzaremmal force-pushed the test-suite branch 2 times, most recently from 0d356d3 to d561c04 Compare September 13, 2025 18:30
@hamzaremmal
chore: add the default entry point to the compiler
74303f1
@hamzaremmal
add the git-hash to the manifest's attributes
09c28cb
@hamzaremmal
chore: fake the reference version to be 3.8.0 for sbt to work correctly
0dce16c 
This should be reverted the minute we have 3.8.0-RC1
@hamzaremmal
hide the access method in :: but allow it to be public in the binary
6dde886
@hamzaremmal
chore: add test configuration for the non-bootstrapped compiler
545fb35
@hamzaremmal
fix: under the new library, scala.Predef has summon in source
9145c90
@hamzaremmal
disable the test until scala#23773 is fixed
5220499
@hamzaremmal
fix: make the order of the keys in the Map deterministic
e55df11 
This was introduced in scala#23769 but it makes the following test fail: https://github.com/scala/scala3/blob/48bc891bd317a62956dc86297feb6e86a73083f6/tests/run/t4954.scala#L8
@hamzaremmal
fix: remove LazyList._empty from the init checker
4e2f3e3 
That `private` field was removed from `scala.collection.immutable.LazyList` in scala@3a84ff3
@hamzaremmal
chore: we are actually running the tasty and tasty cc stlib
247999e
@hamzaremmal
temporarly disable the test suite for explicit nulls
bb35d9c
@hamzaremmal
temporarily disable tests from scala#19084
4966804
@hamzaremmal
fix: adapt tests/neg-custom-args/captures/puretest.scala to the cc …
062ed68 
…library
@hamzaremmal
fix: adapt tests/neg-custom-args/captures/i15772.scala to the cc li…
2bd4920 
…brary
@hamzaremmal
fix: adapt tests/neg-custom-args/captures/use-capset.check to cc st…
394325a 
…dlib
@hamzaremmal
temp: disable test for cc as I believe it shouldn't work in the first…
f69cb45 
… place
@hamzaremmal
ignore BashScriptsTests
7322dbd
@hamzaremmal hamzaremmal marked this pull request as ready for review September 15, 2025 13:41
@hamzaremmal hamzaremmal requested a review from a team as a code owner September 15, 2025 13:41
@hamzaremmal hamzaremmal assigned Gedochao and unassigned hamzaremmal Sep 15, 2025
@hamzaremmal
Copy link
Member Author

hamzaremmal commented Sep 15, 2025 •
edited
Loading

I had a missing comma in the project/Build.scala file. Hence, the amend of the commit and the force push.

@hamzaremmal hamzaremmal merged commit 6c86743 into scala:main Sep 15, 2025
76 of 77 checks passed
@hamzaremmal hamzaremmal deleted the test-suite branch September 15, 2025 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Gedochao Gedochao Gedochao approved these changes

Assignees

@Gedochao Gedochao

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hamzaremmal @Gedochao