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

Set up CI for Scala 3 migration #1

Merged
merged 11 commits into from
Mar 11, 2021
Merged

Set up CI for Scala 3 migration #1

merged 11 commits into from
Mar 11, 2021

Conversation

MaximeKjaer
Copy link

@MaximeKjaer MaximeKjaer commented Mar 4, 2021

Setting up a development environment for incrementally migrating Scio to Scala 3. A few notes:

  • Other CI files have been set to run on release instead of on push. This is to ensure that we just run our migration CI, and not the full set of CI checks. This is crucial in order to be able to make incremental progresss
  • We have added a new CI workflow, migration.yml. Most projects are commented out in this; the idea is to uncomment them as we progress.
  • We have updated the build to use the latest Scala 3 RC release, Scala 3.0.0-RC1.
  • In order to get a single subproject to compile, we have updated the calls to Scala 3.0.0-RC1 TASTy reflection in scio-macros to migrate from 3.0.0-M1 to 3.0.0-RC1
  • Future PRs can address issues on our repo. For instance, see [scio-core] Fix compilation #2

Copy link

@liufengyun liufengyun left a comment

Choose a reason for hiding this comment

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

LGTM, well done 🎉

| found: $tpe
| expected: $resType""".stripMargin
report.throwError(mess)
}

Choose a reason for hiding this comment

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

Are the changes here needed for the CI to be green? If not, I'd suggest delay the change to another PR to fix compilation of core.

Choose a reason for hiding this comment

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

This changes are necessary for the code to compile

@MaximeKjaer MaximeKjaer merged commit 3001fd0 into scala3-master Mar 11, 2021
@MaximeKjaer MaximeKjaer deleted the setup-ci branch March 11, 2021 15:01
vincenzobaz added a commit that referenced this pull request Mar 11, 2021
* Temporarily silence other jobs

* Refactor scala3 version into a variable

* Create ci job testing and compiling each subproject

* Disable fail-fast

* Use RC1 syntax for givens

* Fix isJavaBean

* Fix shadowing of BitSetCoder by private class

* Remove call to showAnsiColored which does not exist anymore

* Fix ambiguous resolution of implicit

* Disable subprojects that don't build

* Disable scio-core

Co-authored-by: vincenzobaz <[email protected]>
vincenzobaz added a commit that referenced this pull request Mar 29, 2021
* Temporarily silence other jobs

* Refactor scala3 version into a variable

* Create ci job testing and compiling each subproject

* Disable fail-fast

* Use RC1 syntax for givens

* Fix isJavaBean

* Fix shadowing of BitSetCoder by private class

* Remove call to showAnsiColored which does not exist anymore

* Fix ambiguous resolution of implicit

* Disable subprojects that don't build

* Disable scio-core

Co-authored-by: vincenzobaz <[email protected]>
vincenzobaz added a commit that referenced this pull request Mar 29, 2021
* Temporarily silence other jobs

* Refactor scala3 version into a variable

* Create ci job testing and compiling each subproject

* Disable fail-fast

* Use RC1 syntax for givens

* Fix isJavaBean

* Fix shadowing of BitSetCoder by private class

* Remove call to showAnsiColored which does not exist anymore

* Fix ambiguous resolution of implicit

* Disable subprojects that don't build

* Disable scio-core

Co-authored-by: vincenzobaz <[email protected]>
MaximeKjaer added a commit that referenced this pull request Apr 12, 2021
* Temporarily silence other jobs

* Refactor scala3 version into a variable

* Create ci job testing and compiling each subproject

* Disable fail-fast

* Use RC1 syntax for givens

* Fix isJavaBean

* Fix shadowing of BitSetCoder by private class

* Remove call to showAnsiColored which does not exist anymore

* Fix ambiguous resolution of implicit

* Disable subprojects that don't build

* Disable scio-core

Co-authored-by: vincenzobaz <[email protected]>
MaximeKjaer added a commit that referenced this pull request Apr 19, 2021
* Temporarily silence other jobs

* Refactor scala3 version into a variable

* Create ci job testing and compiling each subproject

* Disable fail-fast

* Use RC1 syntax for givens

* Fix isJavaBean

* Fix shadowing of BitSetCoder by private class

* Remove call to showAnsiColored which does not exist anymore

* Fix ambiguous resolution of implicit

* Disable subprojects that don't build

* Disable scio-core

Co-authored-by: vincenzobaz <[email protected]>
MaximeKjaer added a commit that referenced this pull request May 5, 2021
* Temporarily silence other jobs

* Refactor scala3 version into a variable

* Create ci job testing and compiling each subproject

* Disable fail-fast

* Use RC1 syntax for givens

* Fix isJavaBean

* Fix shadowing of BitSetCoder by private class

* Remove call to showAnsiColored which does not exist anymore

* Fix ambiguous resolution of implicit

* Disable subprojects that don't build

* Disable scio-core

Co-authored-by: vincenzobaz <[email protected]>
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