Skip to content

Bump Scala 3 to RC1 #10

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

Closed
wants to merge 1 commit into from
Closed

Bump Scala 3 to RC1 #10

wants to merge 1 commit into from

Conversation

anatoliykmetyuk
Copy link
Contributor

This currently fails due to a reworked API of TASTy Reflect:

[error] -- [E006] Not Found Error: /Users/kmetiuk/Projects/scala3/tools/dotty-release-scripts/ecosystem_projects/foo/inspector/src/main/scala/inspector/Inspector.scala:11:24
[error] 11 |    val inspector = new TastyInspector {
[error]    |                        ^^^^^^^^^^^^^^
[error]    |                        Not found: type TastyInspector
[error] -- [E008] Not Found Error: /Users/kmetiuk/Projects/scala3/tools/dotty-release-scripts/ecosystem_projects/foo/inspector/src/main/scala/inspector/Inspector.scala:18:14
[error] 18 |    inspector.inspectTastyFiles(List(tastyFile))
[error]    |    ^^^^^^^^^^^^^^^^^^^^^^^^^^^
[error]    |    value inspectTastyFiles is not a member of Object

inspector project relies on the processCompilationUnit method which was removed. Using TastyInspector object directly without that method results in tests failures (read null instead of some meaningful content). @nicolasstucki any idea how to fix this?

@nicolasstucki
Copy link
Contributor

The API changed. I will update it.

@nicolasstucki
Copy link
Contributor

@anatoliykmetyuk branches should be created in dotty-staging, not in this repo just as we do for dotty.

@nicolasstucki nicolasstucki linked an issue Feb 17, 2021 that may be closed by this pull request
@nicolasstucki
Copy link
Contributor

Replaced with #12

@nicolasstucki nicolasstucki deleted the scala3-bump branch February 17, 2021 08:35
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.

Remove scala3-bump branch from this repo
2 participants