Skip to content
This repository was archived by the owner on Dec 15, 2022. It is now read-only.

Conversation

@smashwilson
Copy link
Contributor

@smashwilson smashwilson commented Nov 27, 2018

Please be sure to read the contributor's guide to the GitHub package before submitting any pull requests.

Requirements

  • Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
  • Suggestion: You can use checklists to keep track of progress for the sections on metrics, tests, documentation, and user research.

Description of the Change

Iterating on our process based on the first round of feedback from the team 😄

Alternate Designs

n/a

Benefits

Documentation more up to date with our current, actual practice. Easier onboarding of new core team members.

Possible Drawbacks

ETOOMUCHMARKDOWN?

Applicable Issues

n/a

Metrics

n/a

Tests

n/a

Documentation

The in-repository core team process guide.

Release Notes

n/a

User Experience Research (Optional)

n/a


View rendered docs/core-team-process.md

smashwilson and others added 2 commits November 27, 2018 15:12
Co-Authored-By: Katrina Uychaco <[email protected]>
Co-Authored-By: Katrina Uychaco <[email protected]>
Co-Authored-By: Tilde Ann Thurium <[email protected]>
Co-Authored-By: Vanessa Yuen <[email protected]>
Copy link

@annthurium annthurium left a comment

Choose a reason for hiding this comment

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

yay, thanks for documenting this!

@smashwilson
Copy link
Contributor Author

Oh right, cc @sguthals and @meaghanlewis for awareness. Mostly I wanted to make sure I was accurately summarizing our discussions from the team 😄

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.06%) to 85.559% when pulling 3567cd1 on aw/process-iteration into 9fe9fcb on master.

Copy link
Contributor

@vanessayuenn vanessayuenn left a comment

Choose a reason for hiding this comment

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

🎉

Copy link
Contributor

@kuychaco kuychaco left a comment

Choose a reason for hiding this comment

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

👍 looks good to me!

curious to see how it is tracking tasks in a project.

Copy link

@drguthals drguthals left a comment

Choose a reason for hiding this comment

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

Looks great! Thank you!

@smashwilson smashwilson merged commit 0bf8106 into master Dec 2, 2018
@smashwilson smashwilson deleted the aw/process-iteration branch December 2, 2018 15:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants