Suggestions for changes

02JanDal / QuickModDoc

Documentation and stuff about QuickMods

2 stars 0 forks source link

Suggestions for changes #2

Open 02JanDal opened 10 years ago

02JanDal commented 10 years ago

A few changes I've been thinking about or that have been suggested:

Remove minified urls (mcf: github: curse: etc.) DONE
- The bandwidth saving really isn't a lot, instead they introduce extra complexity
Selective dependencies
- You need to either install X or Y as dependencies to A
Remove forgeCompat, mcCompat and liteloaderCompat DONE
- Instead have reserved uids (net.minecraftforge, net.minecraft, com.mumfrey.liteloader)
Using URNs?
Remove maven download type DONE
Remove maven dependencies?

stephenmac7 commented 10 years ago

PR #3 fixes the minified urls and maven. I also second removing forgeCompat and liteloaderComat. Don't know about mcCompat though.

02JanDal commented 10 years ago

@ZephireNZ, @luacs1998, @Meegooo: Hey, you seem to be using QMUrls (github:// etc.) in your QuickMod files. They have been removed from the spec, so you should update your QuickMod files. I recommend rawgit.com for replacing github:// urls.

ZephireNZ commented 10 years ago

Thanks for the heads up :)

02JanDal commented 9 years ago

@ZephireNZ: the *Compat fields where removed in 632da0a4df8c91cf8d5047aa509d0863e0a88251, do you think you could fix your QuickMod files? (I often use yours for testing)

ZephireNZ commented 9 years ago

Can do. Will do it once I'm back from work :) On 20/11/2014 5:26 am, "Jan Dalheimer" notifications@github.com wrote:

@ZephireNZ https://github.com/ZephireNZ: the *Compat fields where removed in 632da0a https://github.com/02JanDal/QuickModDoc/commit/632da0a4df8c91cf8d5047aa509d0863e0a88251, do you think you could fix your QuickMod files? (I often use yours for testing)

Reply to this email directly or view it on GitHub https://github.com/02JanDal/QuickModDoc/issues/2#issuecomment-63667409.

ZephireNZ commented 9 years ago

@02JanDal Ok, they should all be removed now. Haven't tested, so there could be some JSON derps or something.

02JanDal commented 9 years ago

@ZephireNZ seems like you forgot to push?

ZephireNZ commented 9 years ago

Whoops, should be pushed now :)

On 22 November 2014 at 05:30, Jan Dalheimer notifications@github.com wrote:

@ZephireNZ https://github.com/ZephireNZ seems like you forgot to push?

Reply to this email directly or view it on GitHub https://github.com/02JanDal/QuickModDoc/issues/2#issuecomment-63995818.