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

More contributing docs #317

Merged

Conversation

ryneeverett
Copy link
Collaborator

Seem commit messages for detail.

I think this is worth a try. If it becomes a problem we can always
revert it. My rationale:

  - This is a good project for newbies to hack on. Nobody's respirator
    has bugwarrior as a dependency. If they break bugwarrior but the
    tests still pass, that's a good opportunity to write more tests.
  - The people who read the contributing docs are going to be better
    contributors than those who don't. Therefore, this is the place we
    want to encourage them to the maximum. I don't *think* it will
    encourage low-effort PR's.
  - When someone's struggling it is probably be easier to review their
    unfinished work early and intervene than to review a monsterous
    changeset they've cludged into working.
I'm assuming a symlink will work here.
@ryneeverett
Copy link
Collaborator Author

I thought of a couple examples as case evidence for WIP PR's:

@ralphbean
Copy link
Sponsor Collaborator

I'm all about it. :) Thanks @ryneeverett! 👍 to merge.

@ryneeverett ryneeverett merged commit 2ea692d into GothenburgBitFactory:develop May 24, 2016
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

Successfully merging this pull request may close these issues.

None yet

2 participants