Automatic native building & latest natives#187
Conversation
…into build-natives
…into build-natives
|
ready for review, I've done all requested changes Job viewable at: https://github.com/sqikerz/lavaplayer/actions/runs/23174458583 |
topi314
left a comment
topi314
left a comment
There was a problem hiding this comment.
this mostly looks good i think with some small exceptions
topi314
left a comment
topi314
left a comment
There was a problem hiding this comment.
I am also unsure if caching the natives works as it should
Ideally we would be caching the final binaries of the action output
does it do that rn?
- merge gitignore files - update action workers - correct publishToMavenLocal to skip natives - cache action output rather than deps
It didn't but it does now. |
|
Cached run: https://github.com/sqikerz/lavaplayer/actions/runs/23385011933 |
|
Need to test running lavaplayer again, give me a moment. |
… variable and skip :natives on JitPack builds
2 participants
yes (also can't lie, thank you very much big bro @claude)
Run can be viewed at:
https://github.com/sqikerz/lavaplayer/actions/runs/23119077532