Jump to content

Addition of Han Chinese to 0AD


Recommended Posts

  On 14/09/2021 at 8:05 AM, Stan` said:

That's the keyword. I didn't review those changes. :) 

Expand  

Yeah, thought so. but maybe you just need to tell everybody that they can commit all the changes they want to their own branch, but that the main branch is restricted.

Github even has that as a nice feature: protected branches

Link to comment
Share on other sites

  On 14/09/2021 at 8:55 AM, Stan` said:

Well yeah. I tend to trust people. But I shouldn't. I hoped stuff could work without micromanagement.

Expand  

Trusting people to try their best is certainly good. Trusting people are born fully fledged coders and knowledgeable of the ins and outs of the project is bound to not work.

The first thing about git is branches are very cheap, commits are very cheap. To test or review I don't need anything committed to master to reduce workload, so there is no need to commit anything to master either.

At one point you will have to look at what was done either way, so commits to master can't reduce your workload only increase it unless you don't care about the master branch in the first place.

Anyway good to run into such issues now, so you can ponder a workflow for in case of migration.

  • Thanks 1
Link to comment
Share on other sites

  On 14/09/2021 at 10:42 AM, Stan` said:

Find me the people and I'm down :D 

Expand  

I know, easier said than done. But I think in the case of this mod (without custom code), everybody who wants the Han included or who already contributed qualifies.

It's more about getting a more "formal" system to not only commit stuff, but also review the stuff other people committed, before it gets included. Even if its not a full review, but just play-testing, this will decrease the amount of bugs that can be introduced.

Cause without a more formal system you find bugs only by accident.

An of course when talking about the complete git migration, there has to be a different workflow and completely different level and quality of reviews.

 

Edited by maroder
Link to comment
Share on other sites

  On 14/09/2021 at 1:22 PM, hyperion said:

Anyway good to run into such issues now, so you can ponder a workflow for in case of migration.

Expand  

Indeed it's nice to run into it now, however the bar for mods has always been lower than the main game. 

  On 14/09/2021 at 1:22 PM, hyperion said:

Trusting people to try their best is certainly good. Trusting people are born fully fledged coders and knowledgeable of the ins and outs of the project is bound to not work.

Expand  

Of course. And then we (or I in that matter) will have to teach them and then you get at the point where it's tempting to do everything yourself because you know you're gonna be there tomorrow and it might faster to do things yourself. I hate doing that and I I believe mistakes are good for learning.  But sometimes it backfires :) 

  On 14/09/2021 at 1:26 PM, maroder said:

 

It's more about getting a more "formal" system to not only commit stuff, but also review the stuff other people committed, before it gets included. Even if its not a full review, but just play-testing, this will decrease the amount of bugs that can be introduced.

Expand  

You need someone with the committment to test. It might take months before that happen. Which can be fine, but also an extremely confusing / frustrating experience.

 

Link to comment
Share on other sites

  On 14/09/2021 at 2:47 PM, Stan` said:

Of course. And then we (or I in that matter) will have to teach them and then you get at the point where it's tempting to do everything yourself because you know you're gonna be there tomorrow and it might faster to do things yourself. I hate doing that and I I believe mistakes are good for learning.  But sometimes it backfires :) 

Expand  

Of course contributors are more of an investment than getting things done quickly. Experienced opensource coders are not the norm. But there is no need for hand holding either, which you tend to fall into all to quickly. :)

Link to comment
Share on other sites

  On 14/09/2021 at 5:27 AM, ymming said:

The application would more be like tomahawk.

Expand  

This is good. Maybe a uniqueish Han trait is their throwy boys are a little shorter range but their archyboys are a little longer range thanks to crossbows.

It appears they definitely lacked slings though. Even the literal slingers.com forums couldn't fine pre 900AD sling.

Link to comment
Share on other sites

  On 14/09/2021 at 2:47 PM, Stan` said:

you get at the point where it's tempting to do everything yourself

Expand  

The perks of maintaining an open source project :D But the teaching aspect is very good for building a community.

  On 14/09/2021 at 2:47 PM, Stan` said:

You need someone with the committment to test. It might take months before that happen. Which can be fine, but also an extremely confusing / frustrating experience.

Expand  

True I guess.

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share

×
×
  • Create New...