-
-
Notifications
You must be signed in to change notification settings - Fork 137
Publish Serial and IO libraries to Maven Central #1421
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
base: main
Are you sure you want to change the base?
Conversation
|
@Stefterv hey good evening, I’ve opened the PR as discussed. Please let me know if any changes are needed Thanks |
| dependencies { | ||
| compileOnly(project(":core")) | ||
| // TODO: https://github.com/java-native/jssc | ||
| implementation(files("library/jssc.jar")) |
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.
#1106 please see here why this was this way
|
@Stefterv hey... saw that TODO was left there in #1106. At the time there wasn't a good public version. Looks like If there's something from the old fork that still needs to be there pls let me knw |
|
Oh great, can you separate that out into a 2nd PR? I will need to look a bit deeper into that and don't want to block this one. |
|
@Stefterv sure you re right, gimme a moment |
|
@Stefterv Done, PR now only has the IO library migration. will open a separate PR for the jssc migration later as you asked. Pls let me know |
|
Ah, please keep the work you did on publishing the serial library, just keep the jssc jar like it was 😅 |
|
@Stefterv oh alright, give me some time college chaos ongoing 🥀 |
|
@avinxshKD lgtm! if @Stefterv is happy with it, let's get this merged in |
Ported Serial and IO libraries from Ant to Gradle with Maven publishing, Serial now uses upstream jssc from Maven Central instead of the bundled fork. This completes the library migration work started in #1407 and #1411.
closes #1403