-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Added monocle. #11468
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
Added monocle. #11468
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like this PR is accidentally reverting many of the other community build project submodules to older commit hashes, which is one of the reasons that the CI is failing.
To fix, you should probably run git submodule update --init
and then git add
those submodules to your commit so they are back on par with current master.
The CI will probably still fail after updating the submodules, but this will help isolate the actual changes needed to get this PR working.
Thanks a lot. It will save me quite some time. |
This currently fail with:
community-build/testOnly dotty.communitybuild.CommunityBuildTestA -- *monocle |
I'm not sure publishing shapeless is going to fix much here, as the build is trying to use shapeless 2 and not shapeless 3. Looking at the upstream Monocle Also, I believe this PR originally had the submodule pointed to a commit hash on the upstream |
Some suggestions: To add the Monocle submodule:
To adjust what's being attempted to be built/tested: modify sbtTestCommand = "coreJVM/test; lawJVM/test; ... etc ...", |
I will check whether the most recent shapeless could be used. thanks for giving a hint |
88d043d
to
051a4ee
Compare
791a0f9
to
a9354ca
Compare
Hey @dotbg, thanks for your contributions! What is the current status of this PR? |
Hi @b-studios, I have just updated the PR. I had few issues with the build before RC2. |
community-build/test/scala/dotty/communitybuild/CommunityBuildTest.scala
Outdated
Show resolved
Hide resolved
Co-authored-by: Tom Grigg <tomegrigg@gmail.com>
Co-authored-by: Tom Grigg <tomegrigg@gmail.com>
Hi @dotbg is this PR ready to review? |
Hi. yes, it is ready to be reviewed |
No description provided.