Skip to content

dev-cmd/tap-new: various tweaks.#10751

Merged
MikeMcQuaid merged 1 commit intoHomebrew:masterfrom
MikeMcQuaid:brew-tap-new-tweaks
Mar 2, 2021
Merged

dev-cmd/tap-new: various tweaks.#10751
MikeMcQuaid merged 1 commit intoHomebrew:masterfrom
MikeMcQuaid:brew-tap-new-tweaks

Conversation

@MikeMcQuaid
Copy link
Copy Markdown
Member

  • --no-git and --branch don't conflict as --branch is still used for the GitHub Actions branch even with --no-git
  • fix some README markdown linting errors
  • fix some YAML/GitHub Actions configuration linting errors

@BrewTestBot
Copy link
Copy Markdown
Contributor

Review period will end on 2021-03-02 at 12:37:07 UTC.

@BrewTestBot BrewTestBot added the waiting for feedback Merging is blocked until sufficient time has passed for review label Mar 1, 2021
Copy link
Copy Markdown
Member

@Rylan12 Rylan12 left a comment

Choose a reason for hiding this comment

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

Looks good to me!

One minor thing: it may be a good idea to reword the --branch flag description to mention that it is also used to create the correct GitHub Actions workflows, even if --no-git is passed.

- `--no-git` and `--branch` don't conflict as `--branch` is still used
  for the GitHub Actions branch even with `--no-git`
- fix some README markdown linting errors
- fix some YAML/GitHub Actions configuration linting errors
@MikeMcQuaid
Copy link
Copy Markdown
Member Author

One minor thing: it may be a good idea to reword the --branch flag description to mention that it is also used to create the correct GitHub Actions workflows, even if --no-git is passed.

Good idea, pushed!

@BrewTestBot BrewTestBot added waiting for feedback Merging is blocked until sufficient time has passed for review and removed waiting for feedback Merging is blocked until sufficient time has passed for review labels Mar 1, 2021
@BrewTestBot
Copy link
Copy Markdown
Contributor

Review period ended.

@BrewTestBot BrewTestBot removed the waiting for feedback Merging is blocked until sufficient time has passed for review label Mar 2, 2021
@MikeMcQuaid MikeMcQuaid merged commit 28bc2b8 into Homebrew:master Mar 2, 2021
@MikeMcQuaid MikeMcQuaid deleted the brew-tap-new-tweaks branch March 2, 2021 13:01
lithammer added a commit to lithammer/homebrew-deadsnakes that referenced this pull request Mar 2, 2021
@BrewTestBot BrewTestBot added the outdated PR was locked due to age label Apr 2, 2021
@Homebrew Homebrew locked as resolved and limited conversation to collaborators Apr 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

outdated PR was locked due to age

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants