Skip to content

Add cross platform tests and rework changelog #41

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 6 commits into from
May 14, 2024
Merged

Conversation

szymon-rd
Copy link
Contributor

@szymon-rd szymon-rd commented Jan 25, 2024

  • Changed latest -> default in tests
  • Added more complete testing infrastructure (for native and js too)

Still WIP, going to add it to gh actions

@szymon-rd szymon-rd mentioned this pull request Jan 25, 2024
@szymon-rd
Copy link
Contributor Author

Blocked by softwaremill/sttp#2082

adpi2 added 3 commits May 8, 2024 16:07
- Reduce the number of args to increase usability
- Test without invoking scala-cli
- Introduce Platform, Config, ChangelogException
- Refactor here and there
@adpi2 adpi2 changed the title Tests and updates Add cross platform tests and rework changelog May 13, 2024
@adpi2 adpi2 force-pushed the tests-n-updates branch from f749ab7 to 43b5cca Compare May 13, 2024 15:12
@adpi2 adpi2 force-pushed the tests-n-updates branch from 43b5cca to 9d702d4 Compare May 13, 2024 15:19
@adpi2
Copy link
Member

adpi2 commented May 13, 2024

@szymon-rd @bishabosha I took this PR over.

The goal of the PR is to test that we can compile and run some toolkit scripts on all different platforms. The tests/shared folder contains the scripts that are valid in all platforms. tests/js, tests/native and tests/jvm contain the script that are only valid for Scala Native, Scala.js and JVM respectively.

I also reworked the changelog code which is responsible for generating the changelog and checking that all version updates are valid. It was not working anymore because of 0.5 being the default Scala Native version in Scala CLI.

@adpi2 adpi2 requested a review from bishabosha May 13, 2024 15:20
@adpi2 adpi2 merged commit 2a054ef into main May 14, 2024
1 check passed
@SethTisue SethTisue deleted the tests-n-updates branch May 21, 2024 15:42
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