-
Notifications
You must be signed in to change notification settings - Fork 46
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
Fix broken links in the guides #763
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, I was in the wrong repo. 😬
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adding two small changes here. I'll merge them and then approve.
Thank you, @mgwalker! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
How many approvals do I need??? |
So, either someone on the @18F/18f-engineering-leadership group or @quarterback have to approve the latest set of changes. |
Bah! I had previously approved them, but I guess it revokes approvals when there are changes now. Approval incoming! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approval number three!
This flow is like if Concur built approval. Just keep approving! Eventually it'll go! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM because no one else has approved yet
Guess we still need the quarterback...this is fun! Next time I won't mash up changes from different guides into one PR (I guess that's better?) |
Changes proposed in this pull request: