-
Notifications
You must be signed in to change notification settings - Fork 32
Consolidate PRs 97 & 100 then run both through current CI #110
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
Consolidate PRs 97 & 100 then run both through current CI #110
Conversation
Maybe you should consider adding |
Rebased & passing. |
Eric, Good idea. I'll try to do it in a separate PR, if I do not run out of time. I am currently working on the one outstanding issue: adding CLA & scalafmt to |
It is just copying that one file from Native to Native.g8 - figured you were just trying to clean up all the PRs. |
You surmised correctly. I am trying to clean up the one Issue and at least get the PRs sorted and minimal. I figure that will make changing an merging easier. Introducing a bit of automation will help also.
Whenever I hear or see the words "just" or "simply", I cower and run for the hills. I figure that I am More seriously, there is only so much complexity that my brain can handle, especially when things The two changes seem to have succeeded. I'm off to see if adding |
f1d472a
to
15d7e6f
Compare
I fixed the broken All Green & ready. |
Reduce accumulated clutter: Consolidate and superseded PRs 97 & 100 then run through current CI.
This should make it easier for both humans and scala-steward to figure out the current & pending states.
The superseded PRs can be closed after this PR is merged.