Skip to content
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

Separate codebases #9

Open
appgurueu opened this issue Oct 25, 2019 · 18 comments
Open

Separate codebases #9

appgurueu opened this issue Oct 25, 2019 · 18 comments

Comments

@appgurueu
Copy link
Contributor

This repo is a collection of different codebases, and as such, poorly manageable. Bugs stack up and we need to port every fix for MTG / factions etc here manually. So I propose that we:

  • Separate this codebase from MTG
  • Separate it into many small mods and make the PK org have multiple subrepos
  • Separate other mods from it, and don't apply changes directly in the code but using extension mods afterwards

This will mean quite some work at first, but will be extremely useful in the long term. We are running an outdated MTG with tons of outdated mods with many tweaks and fixes by us poured in, making it hard to merge.

The benefits will be great: Straightforward licensing, less work, easy updates, merging, new features, and finally being able to concentrate on our neatly separated PK code.

@archfan7411
Copy link
Contributor

Overall agreed, the prospect of separating custom(ized) mods into other PK repos is especially alluring. Unsure what you mean by "Separate this codebase from MTG" though, could you elaborate?

@appgurueu
Copy link
Contributor Author

kingdoms_game extends minetest_game, so we should separate it

@appgurueu appgurueu pinned this issue Oct 26, 2019
@archfan7411
Copy link
Contributor

How would that be accomplished?

@appgurueu
Copy link
Contributor Author

By having the MTG mods separated from this mods. You would always get the freshest ones by pulling them. Texmex is one example of a provider of freshly chopped MTG mods

@appgurueu
Copy link
Contributor Author

I'm not sure about the details, though. We need a way to manage a collection of mods as one game. I'd most likely go with submodules. But apparently there have been issues with that...

@appgurueu
Copy link
Contributor Author

Other members: what do you think?
Pings in no specific order:

@luk3yx
Copy link
Contributor

luk3yx commented Oct 26, 2019

I suggest something similar to https://github.com/BlockySurvival/bls_mods/ (including the instructions on how to clone and update the repository).

@appgurueu
Copy link
Contributor Author

Indeed, that's simply awesome. Conclusions:

  • Separate game and modpack, make a PK modpack
  • Use submodules
  • Proper Git instructions
  • Steal their repo

@appgurueu
Copy link
Contributor Author

In general, BlockySurvival is a great example for how the entire organization could work. Amazing.

@ghost
Copy link

ghost commented Oct 26, 2019

Hola, I am late to the party.

@ghost
Copy link

ghost commented Oct 27, 2019

Hmmmmm.

@appgurueu
Copy link
Contributor Author

First step done ! See our brand new Minetest Game Modpack 🎉

@appgurueu
Copy link
Contributor Author

Second step done ! Kingdoms Game repo made (uses submodules as well)

@appgurueu
Copy link
Contributor Author

Ideally, the final step, as I imagine it (probably by the end of the year), will be swapping out Persistent Factions by the Kingdoms Game, keeping the world. @archfan7411

@archfan7411
Copy link
Contributor

Awesome! Just one minor thing: the games are minetest_game and kingdoms_game but you used dashes in place of underscores. Obviously very minor, it just bugs my OCD. Optional if you want to change it.

@appgurueu
Copy link
Contributor Author

Yeah, the thing is, GitHub convention pretty much uses hyphens, as perceived by me. So I just adhere to it and use a-b-c instead of a_b_c. Different naming conventions. I would prefer the GH one as it's more visually balanced, the hyphen being in the middle of the line instead of at the bottom (it just looks neater, doesn't it?). At least no lowerCamelCase.

@archfan7411
Copy link
Contributor

At the risk of bikeshedding, minetest_game is "correct" because that's the actual name of the game :P
This is getting offtopic though, so I'll just drop it. Thanks again for going through all that work to make the repos!

@appgurueu
Copy link
Contributor Author

Change made.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants