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

Add support for Scala Native 0.4 #482

Merged
merged 8 commits into from
Jan 29, 2021
Merged

Conversation

lolgab
Copy link
Contributor

@lolgab lolgab commented Jan 23, 2021

Fixes: #282

@lolgab lolgab force-pushed the support-scala-native branch 7 times, most recently from a8fd9c9 to 079be9c Compare January 23, 2021 19:06
@SethTisue
Copy link
Member

I don't know Scala Native stuff, but this looks fine to me for merge after rebase 🤷

@lolgab do you feel there is any need for additional review here?

@lolgab lolgab force-pushed the support-scala-native branch from 217c467 to c805f48 Compare January 29, 2021 08:21
@lolgab
Copy link
Contributor Author

lolgab commented Jan 29, 2021

@SethTisue It sounds right to me and the tests pass, but let's ask @WojciechMazur if he can :)

@lolgab lolgab force-pushed the support-scala-native branch from c805f48 to 047e953 Compare January 29, 2021 09:08
Co-authored-by: Sébastien Doeraene <[email protected]>
@lolgab lolgab requested a review from sjrd January 29, 2021 15:08
@SethTisue
Copy link
Member

SethTisue commented Jan 29, 2021

Would it be hard to also update the CircleCI config?

I don't really know where we stand with CircleCI. @ashawley added it as "experiment" and "backup" at #324

Personally, I'm fine with Travis-CI, or perhaps GitHub Actions in the future. Normally having two CI systems around doesn't really cost us anything.... except times when it does, like now. Or perhaps it would actually be simple to update?

@lolgab lolgab force-pushed the support-scala-native branch from 7b7f3a3 to e5f4e51 Compare January 29, 2021 16:16
@lolgab
Copy link
Contributor Author

lolgab commented Jan 29, 2021

@SethTisue Added a CircleCI job to test on Scala Native

@SethTisue SethTisue merged commit 7af5889 into scala:master Jan 29, 2021
@SethTisue
Copy link
Member

thank you!!

@SethTisue
Copy link
Member

as for publishing it, #484

@@ -0,0 +1 @@
version = "2.7.4"
Copy link
Contributor

Choose a reason for hiding this comment

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

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.

Scala Native Support
5 participants