Fork me on GitHub
#clojure-uk
<
2019-02-15
>
dominicm06:02:09

Phabricator had this as the default

dominicm06:02:05

Although this seems way lighter

agile_geek08:02:06

Bore da pawb welsh_flag

thomas08:02:22

hmmmm... I read that first as 'daft' pull request and wondered what it was facepalm

😂 5
Conor09:02:18

I too love to solve social problems with technology 😛

otfrom09:02:49

putting WIP in the title of the PR was far too difficult

danm10:02:27

Heh. We don't use PRs, just do all work on master and require commits to be non-breaking, but I suspect that would fall down with a more open project and/or something that was more of a monolithic app instead of lots of small microservices

alexlynham10:02:25

my last project at digital was all trunk-based and it completely confirmed my suspicions about trunk

😅 5
danm10:02:23

What were those suspicions? I take it you didn't like it? 😉

alexlynham10:02:37

no, that it's the best way to speed up delivery

danm10:02:49

Oh cool :) I agree but I’ve tended to find pushback from others outside of our and a few other teams.

mccraigmccraig10:02:06

we do multi-trunk - a trunk branch for each release cycle, with the general principle that all lower-release trunks should be merged into a higher-release trunk. it generally works pretty nicely, although it does get hairy if a release is delayed for a while for some reason and you consequently have more than one very active trunk

danm10:02:07

Though thinking about it, the pushback is generally from people who haven’t tried it...

rickmoynihan12:02:41

I don’t think there’s any objectively better git workflow; it entirely depends on what is right for the team and how they’re organised and managed. Trunk based dev at scale complects the workflow to force integration at the expense of isolation. I’d say it’s the easiest (not necessarily best) way to speed up delivery in the short term. It’s the singleton pattern from OO as a workflow, which is fine when you only want one system; but if you need many you’ll find people coordinating adhoc outside of the version control system and CI. For example with trunk based dev, how do you integrate large scale changes across many services/repos without shutting down the shop floor? You trade off that capability; which branching workflows enable (with more complex supporting infrastructure).

alexlynham12:02:40

you feature flag and have a lot of redundant code is what I found

alexlynham12:02:55

but then once the flags reverse you can prune it all pretty aggressively

alexlynham12:02:47

I found that the QA effort, the having good test/dev data effort and writing good tests effort went up, but the brainpower required to dev and get code out was an order of magnitude better

rickmoynihan13:02:08

> you feature flag :thumbsup: yes you can — but then you need to manage the different sets of flags across services etc. Ultimately you’re moving organisational problems into code and increasing complexity of things to test. Which is totally fine, so long as you’re aware of what you’re doing. Feature toggles often just give the illusion of integration; until you try a combination you thought would work and realise they’re incompatible.

rickmoynihan13:02:21

At the end of the day I think mandating any one workflow without consideration is removing tools from your shed. (Not saying anyone here is doing this btw) You can make more subtle trade offs still by combining feature toggles with branches for example.

rickmoynihan13:02:39

I think there’s only been one word I disagreed in this conversation and that was the word best 🙂

rickmoynihan14:02:22

I should also say we do use feature toggles for this in some of our services… (though we still use branches too). And they can work well, in particular it lets you decomplect notions of merged & integrated. For example one of our customers sponsored a large complex feature. It’s not generalised to everyone else, so feature toggling made sense, as it meant we can pick up cross cutting improvements in particular styling changes etc. However I’m under no illusion that the feature isn’t being tested with some other features yet… some combinations may be assumed to work but untested etc. It’s a JTBD to unpick that, and test combinations of features. We use duct, which is pretty good at this sort of thing… Thinking about it I recon we can probably represent those features in the keyword hierarchy, so we can use that to represent what is expected to work and then test those combinations explicitly.

rickmoynihan10:02:49

we’ve had a ready for review tag for years - to indicate readiness — so whilst it’s a nice feature and has a few more protections it feels like it’s an overly specified feature. Main advantage I guess is it encourages more people into better default workflows. We’ll certainly be using it, but I find it hard to believe there weren’t more important features of equivalent complexity to add.

danm10:02:35

Yeah, we apparently used phabricator back before I joined

danm10:02:50

Although at the time I think we might have mostly been an svn shop too

maleghast11:02:31

isn't Phabricator a PHP groupware for devs product..?

dominicm11:02:05

What is groupware

maleghast11:02:16

Software for groups that provides a suite of facilites / apps that allow that group to work, so messaging, calendaring, planning, documenting (wiki) etc.

dominicm11:02:09

I think so, yeah

maleghast12:02:54

I had a colleague in Manila who was keen to move us over to it instead of using Atlassian and Github... $20/seat (up to 50 seats then more for no more $s) seems pricey, but when you add it up... Thing is I really like Clubhouse and I really like Github and I hate Wikis anyway, so I'd rather not have one...

maleghast12:02:09

(apart from Wikipedia, that one can stay)

maleghast12:02:44

Quick announcement / thank you to you lot - I have hired someone! He signed the contract yesterday and starts on March 4th, so I am excited to soon be un-alone... I am still hiring on the Front End (see #jobs and Functional Works), so if you know of any young / new-to-ClojureScript (age is not a factor) devs who want to change the World for good, please send them in my direction... 😉

👍 35
mccraigmccraig12:02:52

congratulations!

mccraigmccraig12:02:54

the first step on the path to you never writing any code 😉

maleghast12:02:42

From your mouth to God's(*) ears... 😉 (*clearly I am an atheist, so this is a silly saying)

maleghast12:02:52

and thanks 🙂

jasonbell14:02:28

The wisdom of experience 🙂

folcon13:02:43

Morning, and congrats @maleghast! :)…