-
-
Notifications
You must be signed in to change notification settings - Fork 12.5k
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
action-docs 2.5.0 (new formula) #195773
action-docs 2.5.0 (new formula) #195773
Conversation
85f9598
to
54555aa
Compare
54555aa
to
1b7b97f
Compare
|
1b7b97f
to
1f70137
Compare
Thanks @chenrui333. PR checks are all green now. Please re-review when you get a chance. |
1f70137
to
5455a74
Compare
5455a74
to
fcbf53c
Compare
@chenrui333 thank you for the review. I see that you approved and then force pushed which then removed the approval. Would you mind reviewing again please? Edit: it says this
Perhaps I should force push a new commit and then you can rereview? |
Signed-off-by: nitrocode <[email protected]> action-docs: update test Signed-off-by: Rui Chen <[email protected]> use yaml for heredoc delimiters Signed-off-by: Rui Chen <[email protected]> Signed-off-by: nitrocode <[email protected]>
fcbf53c
to
f170e5e
Compare
we dont need commit fresh, I was just making some test code style changes, but it should be all good. thanks @nitrocode. |
Thanks! |
HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>
, where<formula>
is the name of the formula you're submitting?brew test <formula>
, where<formula>
is the name of the formula you're submitting?brew audit --strict <formula>
(after doingHOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>
)? If this is a new formula, does it passbrew audit --new <formula>
?