-
Notifications
You must be signed in to change notification settings - Fork 126
updated sbt to 1.5.1, updated deprecated 0.13 config style #311
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
Conversation
Thanks for the pr @domizei385! Let me verify this locally with the sbt plugin, but this looks good. |
Everything seems to be working fine when testing the sbt plugins, except for the js tests
Any ideas? |
Not yet. Will check later tonight. Unless you beat me to it. |
Sounds good. I'll see if I can figure it out. If not, I'll push up a branch to the sbt plugin repo and tag you on it. |
Okay. Locally I got all tests to work, but I have messed with the plugin quite a lot (when I was creating a build for my customer to support 2.12.13) so I need to prepare it before I can create a public PR. |
Yes I do. I have a pr sent up there that has everything passing but the js tests. |
👍 I’ll create the PR in a few hours. Then we can cherry pick the best pieces from both PRs. |
Please see scoverage/sbt-scoverage#324 for related sbt plugin PR |
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.
Just a quick question on Scala js but other than that, LGTM
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.
Actually I just got some clarification that the sjs1.5
isn't a thing, so we'll need to switch this to just be sjs1
.
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.
Alright verified that all is working locally now with _sjs1
.
hopefully fix sjs artifact name