Stan` Posted September 14, 2021 Report Share Posted September 14, 2021 Well I suppose that's because gather_grain is missing from all of them --' 1 Quote Link to comment Share on other sites More sharing options...
maroder Posted September 14, 2021 Report Share Posted September 14, 2021 But I guess that isn't intended? Quote Link to comment Share on other sites More sharing options...
Stan` Posted September 14, 2021 Report Share Posted September 14, 2021 On 14/09/2021 at 7:08 AM, maroder said: But I guess that isn't intended? Expand No there have been so many changes in the mod that I can't keep track. Some of them were good and the others were pretty weird or bad. 1 Quote Link to comment Share on other sites More sharing options...
wowgetoffyourcellphone Posted September 14, 2021 Report Share Posted September 14, 2021 On 14/09/2021 at 7:09 AM, Stan` said: No there have been so many changes in the mod that I can't keep track. Some of them were good and the others were pretty weird or bad. Expand Isn't that your working copy? XD Quote Link to comment Share on other sites More sharing options...
Stan` Posted September 14, 2021 Report Share Posted September 14, 2021 Other people have access now... Quote Link to comment Share on other sites More sharing options...
Ceres Posted September 14, 2021 Report Share Posted September 14, 2021 Where? Quote Link to comment Share on other sites More sharing options...
maroder Posted September 14, 2021 Report Share Posted September 14, 2021 On 14/09/2021 at 7:09 AM, Stan` said: Some of them were good and the others were pretty weird or bad. Expand Isn't that the reason for the git-flow and branch logic? Nothing makes it into the main branch without review. @Ceres the github repo: https://github.com/0ADMods/han_china 1 Quote Link to comment Share on other sites More sharing options...
Ceres Posted September 14, 2021 Report Share Posted September 14, 2021 (edited) Ok, that's the one I had looked into, so I was on the right path. Edited September 14, 2021 by Ceres Quote Link to comment Share on other sites More sharing options...
hyperion Posted September 14, 2021 Report Share Posted September 14, 2021 On 14/09/2021 at 7:42 AM, maroder said: Isn't that the reason for the git-flow and branch logic? Nothing makes it into the main branch without review. Expand To used to to svn 1 Quote Link to comment Share on other sites More sharing options...
Stan` Posted September 14, 2021 Report Share Posted September 14, 2021 On 14/09/2021 at 7:42 AM, maroder said: Nothing makes it into the main branch without review. Expand That's the keyword. I didn't review those changes. Quote Link to comment Share on other sites More sharing options...
maroder Posted September 14, 2021 Report Share Posted September 14, 2021 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 Quote Link to comment Share on other sites More sharing options...
Stan` Posted September 14, 2021 Report Share Posted September 14, 2021 Well yeah. I tend to trust people. But I shouldn't. I hoped stuff could work without micromanagement. 1 Quote Link to comment Share on other sites More sharing options...
maroder Posted September 14, 2021 Report Share Posted September 14, 2021 On 14/09/2021 at 8:55 AM, Stan` said: But I shouldn't. Expand Nah, that's a good trait, you should. But that's no reason not to have a system that can minimize possible mistakes. But yes some kind of micromanagement is always necessary I guess. Possibly something you could delegate? I.e some form of community reviews. Quote Link to comment Share on other sites More sharing options...
Stan` Posted September 14, 2021 Report Share Posted September 14, 2021 So by trusting people ? 1 Quote Link to comment Share on other sites More sharing options...
maroder Posted September 14, 2021 Report Share Posted September 14, 2021 (edited) Yeah, but spread the trust between many people. Checks and balances. Nothing gets committed (to main) without review of someone. More eyes are always better. Edited September 14, 2021 by maroder 1 Quote Link to comment Share on other sites More sharing options...
Stan` Posted September 14, 2021 Report Share Posted September 14, 2021 On 14/09/2021 at 9:57 AM, maroder said: Checks and balances. Nothing gets committed (to main) without review of someone. More eyes are always better Expand Find me the people and I'm down Quote Link to comment Share on other sites More sharing options...
Ceres Posted September 14, 2021 Report Share Posted September 14, 2021 Divide et impera! Quote Link to comment Share on other sites More sharing options...
Stan` Posted September 14, 2021 Report Share Posted September 14, 2021 On 14/09/2021 at 10:54 AM, Ceres said: Divide et impera! Expand I thought that was for your enemies? Here I'm trying to build something :p 1 Quote Link to comment Share on other sites More sharing options...
Ceres Posted September 14, 2021 Report Share Posted September 14, 2021 Not Han dynasty, but maybe nevertheless inspiring: A Brief Introduction to Ancient Chinese Military Weapons https://www.viewofchina.com/ancient-chinese-military-weapons/ Quote Link to comment Share on other sites More sharing options...
hyperion Posted September 14, 2021 Report Share Posted September 14, 2021 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. 1 Quote Link to comment Share on other sites More sharing options...
maroder Posted September 14, 2021 Report Share Posted September 14, 2021 (edited) On 14/09/2021 at 10:42 AM, Stan` said: Find me the people and I'm down 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 September 14, 2021 by maroder Quote Link to comment Share on other sites More sharing options...
Stan` Posted September 14, 2021 Report Share Posted September 14, 2021 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. Quote Link to comment Share on other sites More sharing options...
hyperion Posted September 14, 2021 Report Share Posted September 14, 2021 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. Quote Link to comment Share on other sites More sharing options...
MoLAoS Posted September 14, 2021 Report Share Posted September 14, 2021 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. Quote Link to comment Share on other sites More sharing options...
maroder Posted September 14, 2021 Report Share Posted September 14, 2021 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 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. Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.