generated from hackforla/.github-hackforla-base-repo-template
-
-
Notifications
You must be signed in to change notification settings - Fork 27
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
Add missing documentation to the PR review guide #419
Labels
complexity: small
All steps are laid out in detail so that someone new to the project can work on it
dependency
Issue has dependencies
feature: docs: PD team documentation
documentation on PD team processes and architecture, etc.
ready for dev lead
ready for product
role: back end
role: dev
role: product
s: PD team
stakeholder: People Depot Team
size: 0.25pt
Can be done in 0.5-1.5 hours
Milestone
Comments
shmonks
added
draft
This issue is not fully-written
feature: docs: PD team documentation
documentation on PD team processes and architecture, etc.
and removed
feature: missing
labels
Oct 17, 2024
This comment was marked as outdated.
This comment was marked as outdated.
This was referenced Oct 18, 2024
shmonks
added
s: PD team
stakeholder: People Depot Team
size: 0.25pt
Can be done in 0.5-1.5 hours
and removed
draft
This issue is not fully-written
milestone: missing
stakeholder: missing
size: missing
labels
Oct 24, 2024
shmonks
added
complexity: small
All steps are laid out in detail so that someone new to the project can work on it
role: dev
role: back end
and removed
points: missing
complexity: missing
role: missing
labels
Oct 24, 2024
shmonks
added
dependency
Issue has dependencies
and removed
good first issue
Good for newcomers
labels
Oct 25, 2024
This comment was marked as resolved.
This comment was marked as resolved.
shmonks
moved this from 🧊Ice Box
to 🏗In progress-actively working
in P: PD: Project Board
Oct 28, 2024
shmonks
moved this from 🏗In progress-actively working
to 📋Prioritized Backlog
in P: PD: Project Board
Oct 31, 2024
OverviewWe need to create a list of common things to look out for when doing a PR review, so that developers can catch things which can often be missing/wrong Action Items
Resources/Instructions/Tags1.01 Example of another team's PR review guide Website team's PR review guide Documentation Additional NotesWhere would this information have been useful?Which roles will benefit most from this information?
|
vanessaavviles
moved this from 📋Prioritized Backlog
to 🏗In progress-actively working
in P: PD: Project Board
Oct 31, 2024
shmonks
moved this from 🏗In progress-actively working
to 🧊Ice Box
in P: PD: Project Board
Nov 14, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
complexity: small
All steps are laid out in detail so that someone new to the project can work on it
dependency
Issue has dependencies
feature: docs: PD team documentation
documentation on PD team processes and architecture, etc.
ready for dev lead
ready for product
role: back end
role: dev
role: product
s: PD team
stakeholder: People Depot Team
size: 0.25pt
Can be done in 0.5-1.5 hours
Dependency
Overview
We need to make it easy to:
This issue is the place to record missing documentation and start creating related new issues.
Details
see the following issues instead
Action Items
feature: docs: PD team documentation
size: 0.25pt
s: PD team
ready for dev lead
Resources/Instructions
Section 2: List of comments/issues about missing documentation for PR Review guide
The text was updated successfully, but these errors were encountered: