dblock / dblock/code.dblock.org

Forks vs. branches?

Open
#65 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

post topic suggestion
Dominant language
JavaScript
Stars
7
Forks
30
PR merge metrics
No merged PRs in 30d

Description

Screen Shot 2019-03-13 at 11 46 37 AM

Yeah, we’ve not seen more/fewer problems with conflicts – our rationale has always been tools-based. I did recently write a Peril rule to enforce this here: https://github.com/artsy/peril-settings/pull/98 - Ash

I'm on the branch train 😄 . I know part of the reason for forks is it's harder to accidentally merge to master. -Justin

It’s interesting because that’s the reason mobile repos still don’t use forks – because sharing secrets (required for CI, etc) is difficult to do on OSS repos.

orta: branch train here, I always am checking out other people’s branches and improving
but I also don’t care about leaving stay branches around etc
( but I build tools to automate that so others don’t get annoyed )
but the biggest positive is that you can use secure env vars in CI for branches
so you can make better infra

alloy:
Branches 🚋 as well. My branch work shouldn’t be hidden from coworkers in general, especially not somewhere in the future when I may delete my fork or leave the co. It also makes collaborating on branches with coworkers easier through GitHub.

daniel:
I’m very pro branches, we set up a processs at my old gig where stale branches are shamelessly culled after a long period of time. I think branches lowers the barrier to entry for pairing significantly
Also, from an IP perspective. I think it’s a bit weird to have company work on a repo that isn’t technically owned by the co - just seems a bit weird to me

re-read https://artsy.slack.com/archives/C03J4L2KK/p1552490693081200

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

No repository file or test is identified. Start by reviewing the linked Artsy workflow article, the Peril pull request, and the referenced Slack discussion to determine whether this should become a documented decision; done would require a clearly scoped documentation change and an agreed conclusion.

Written by the indexing model from the issue text.

Assessment

Tech stack
git, github
Domain
developer-experience, documentation
Issue type
Documentation
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.