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

Fix broken links in the guides #763

Merged
merged 10 commits into from
Dec 11, 2024
Merged

Fix broken links in the guides #763

merged 10 commits into from
Dec 11, 2024

Conversation

michelle-rago
Copy link
Contributor

Changes proposed in this pull request:

  • These broken links were noted in a Siteimprove report.
  • A few of the commits have a comment where some explanation might be helpful.

quarterback
quarterback previously approved these changes Dec 4, 2024
mgwalker
mgwalker previously approved these changes Dec 4, 2024
Copy link
Contributor

@mgwalker mgwalker left a 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. 😬

@mgwalker mgwalker dismissed their stale review December 4, 2024 17:29

I didn't actually review it yet

Copy link
Contributor

@mgwalker mgwalker left a 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.

mgwalker
mgwalker previously approved these changes Dec 4, 2024
@michelle-rago
Copy link
Contributor Author

Thank you, @mgwalker!

Copy link
Contributor

@jasnakai jasnakai left a comment

Choose a reason for hiding this comment

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

LGTM!

@michelle-rago
Copy link
Contributor Author

How many approvals do I need???

@beechnut
Copy link
Contributor

Waiting on code owner review from 18F/18f-engineering-leadership and/or quarterback.

So, either someone on the @18F/18f-engineering-leadership group or @quarterback have to approve the latest set of changes.

@mgwalker
Copy link
Contributor

Bah! I had previously approved them, but I guess it revokes approvals when there are changes now. Approval incoming!

Copy link
Contributor

@mgwalker mgwalker left a comment

Choose a reason for hiding this comment

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

Approval number three!

@mgwalker
Copy link
Contributor

This flow is like if Concur built approval. Just keep approving! Eventually it'll go!

Copy link
Contributor

@cantsin cantsin left a 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

@michelle-rago
Copy link
Contributor Author

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?)

@michelle-rago michelle-rago merged commit 24eef24 into main Dec 11, 2024
8 checks passed
@michelle-rago michelle-rago deleted the mr/link-fixes branch December 11, 2024 21:33
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.

7 participants