Skip to content

Conversation

@abristow3
Copy link

Initial release for the Flux Clan Plugin.

A comprehensive clan management plugin designed for the Flux OSRS clan community. This plugin provides real-time event tracking, competition leaderboards, and clan information directly in your RuneLite client.

Initial release for the Flux Clan Plugin
@runelite-github-app
Copy link

runelite-github-app bot commented Oct 16, 2025

@abristow3
Copy link
Author

Hi, I am having issues resolving my CI failure. I am running these commands to try and remedy it and get the matching checksums:

./gradlew clean
./gradlew --write-verification-metadata sha256,pgp help

And then pushing to my repo the changes, and updating the commit hash in my merge request here.
I am a bit unfamiliar with Gradle, is there something I am doing incorrect or am unaware of? Thank you for any and all help

@tylerwgrass
Copy link
Member

You are adding a bunch of new dependencies that are not verified. All new dependencies will need to be verified and if you are adding that many the chances of your plugin being approved are ~0.
You can likely fix most of them by replacing with approved options listed here:

because "afterdark-pvm-loot-tracker"

@riktenx
Copy link
Member

riktenx commented Oct 16, 2025

I would take it a step further and say that if you're adding even one moderately-sized new dependency (as in, no other hub plugin or RuneLite itself has previously used it) the chance of your plugin getting reviewed is basically zero.

@abristow3
Copy link
Author

I am fine using existing version, am very new at this. I don't think I am using anything that isn't already being used such as JSON, HTTPClient, GoogleSheets, etc. just a version mismatch I guess based on the approved list

@runelite-github-app
Copy link

runelite-github-app bot commented Oct 16, 2025

This plugin requires a review from a Plugin Hub maintainer. The reviewer will request any additional changes if needed.


Internal use only: Reviewer details Maintainer details

@abristow3
Copy link
Author

Okay I think I was able to address the issues with the dependencies at least. Looks like I need to address using sout and serr

@abristow3
Copy link
Author

I think I was able to resolve all the CI issues. Thank you all for assistance

@abristow3
Copy link
Author

Good morning, I just wanted to see if there is anything else I need to do / provide for this PR. Thank you guys for all you do!

@DominickCobb-rs
Copy link
Contributor

Reviews for new hub plugins can take a while. It's not uncommon to wait more than two weeks. When someone reviews they'll let you know if anything needs a change.

@abristow3
Copy link
Author

Reviews for new hub plugins can take a while. It's not uncommon to wait more than two weeks. When someone reviews they'll let you know if anything needs a change.

Ok I see! Thank you for the information 🙂

@abristow3 abristow3 closed this Oct 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants