Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Add suggestion to use Allstar to enforce consistency across repos #321

Merged
merged 3 commits into from
Oct 3, 2022
Merged

Add suggestion to use Allstar to enforce consistency across repos #321

merged 3 commits into from
Oct 3, 2022

Conversation

markdboyd
Copy link
Contributor

No description provided.

@svenaas svenaas requested a review from kbighorse September 23, 2022 16:02
@svenaas
Copy link

svenaas commented Sep 23, 2022

This looks good to me. @Sgtpluck, you've done a fair bit of work in this file. Would you like to review as well?

Copy link
Contributor

@Sgtpluck Sgtpluck left a comment

Choose a reason for hiding this comment

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

I think it looks good! Is it possible to add a link to where cloud.gov is using it, @markdboyd? That would be very nice to have!

@markdboyd
Copy link
Contributor Author

@Sgtpluck How about 58e6ee4?

@Sgtpluck
Copy link
Contributor

@markdboyd perfect!!! thank you.

@Sgtpluck
Copy link
Contributor

The pa11y failure is fine as per actions/first-interaction#10 (comment) -- it's because the PR is coming from a fork.

@markdboyd
Copy link
Contributor Author

@Sgtpluck OK, thanks! I don't have access to merge PRs on this repo, so someone else will have to merge for me

@svenaas
Copy link

svenaas commented Sep 26, 2022

I have merge permission; I was just giving @kbighorse a chance to review too before we press the button.

Copy link
Contributor

@kbighorse kbighorse left a comment

Choose a reason for hiding this comment

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

👍🏾

@kbighorse kbighorse merged commit 32ef570 into 18F:main Oct 3, 2022
@markdboyd markdboyd deleted the add-allstar-suggestion branch October 4, 2022 14:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants