JDA-Utilities icon indicating copy to clipboard operation
JDA-Utilities copied to clipboard

Fix Gradle File (Probably)

Open ithinkimcats opened this issue 2 years ago • 3 comments

Pull Request

Pull Request Checklist

Please follow the following steps before opening this PR.
PRs that do not complete the checklist will be subject to denial for missing information.

Pull Request Information

Check and fill in the blanks for all that apply:

  • [X] My PR fixes a bug, error, or other issue with the library's codebase.
  • [ ] My PR is for the ______ module of the JDA-Utilities library.
  • [ ] My PR creates a new module for the JDA-Utilities library: ______.

Description

Describe your PR here.

This should fix the Gradle file (maybe) but I also don't know anything about Gradle or Java so I don't know really. I also don't know how to have it "publish to dv8's host" but I have tried (?) to make it do that based on Gradle documentation. I have no idea if this works how you intended it in java cup but when I test it, it does produce the jars and the poms and it says "BUILD SUCCESSFUL", but I really do not know! Yeah.

ithinkimcats avatar Feb 09 '22 09:02 ithinkimcats

This Looks Like Code

SplitPixl avatar Feb 09 '22 09:02 SplitPixl

I think this should be it.

ithinkimcats avatar Feb 10 '22 07:02 ithinkimcats

I've suggested (and highly encouraged) in https://github.com/Chew/JDA-Chewtils/issues/53 to just simply rewrite the damn build.gradle file (bonus: a rewrite in kotlin). This was written for an ancient version of Gradle and the entire DSL has significantly changed. This file can be heavily reduced with modern features, and I think it's time to stop dragging this dying horse to its knees. The code does work, yeah, but anyone who uses and knows (or even works for) Gradle cannot help but be abhorred by this poor file.

If you really want, though, my build.gradle does support pushing to a maven repo, in an ass-backwards way as you'd expect, given the file I've been given.

Also, if you're gonna bump the Gradle version, be sure to run gradle wrapper --gradle-version=7.4

Chew avatar Feb 12 '22 00:02 Chew