Skip to content
This repository has been archived by the owner on Aug 3, 2023. It is now read-only.

feat(docs): add CONTRIBUTING.md #268

Merged
merged 2 commits into from
Jul 29, 2019
Merged

feat(docs): add CONTRIBUTING.md #268

merged 2 commits into from
Jul 29, 2019

Conversation

ashleygwilliams
Copy link
Contributor

No description provided.

CONTRIBUTING.md Show resolved Hide resolved
CONTRIBUTING.md Show resolved Hide resolved
CONTRIBUTING.md Show resolved Hide resolved
CONTRIBUTING.md Show resolved Hide resolved
CONTRIBUTING.md Show resolved Hide resolved
CONTRIBUTING.md Show resolved Hide resolved
CONTRIBUTING.md Show resolved Hide resolved
@xortive
Copy link
Contributor

xortive commented Jun 24, 2019

This might be too much of a maintenance burden, but I think it would be nice if we had a small section on "How to work on wrangler" with like a very high-level map of the codebase, and how to build/install a local copy of wrangler.

@ashleygwilliams
Copy link
Contributor Author

@granjef3 i'm super down for that! if you open an issue i think that'd be great

@ashleygwilliams ashleygwilliams changed the title [WIP] feat(docs): add CONTRIBUTING.md feat(docs): add CONTRIBUTING.md Jul 3, 2019
CONTRIBUTING.md Outdated
received all required reviews.
- All PRs should be labelled with a changelog label: BREAKING, Feature, Bug, Maintenance, Docs

### Reviewing
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this section TODO or should we remove the heading?

@ashleygwilliams ashleygwilliams added regression Something is broken, but works in previous releases status - changes requested labels Jul 15, 2019
@ashleygwilliams ashleygwilliams merged commit 5729c27 into master Jul 29, 2019
@delete-merged-branch delete-merged-branch bot deleted the contrib branch July 29, 2019 15:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
regression Something is broken, but works in previous releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants