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

audit: run syntax check for new formulae earlier #177439

Closed
wants to merge 1 commit into from
Closed

Conversation

iMichka
Copy link
Member

@iMichka iMichka commented Jul 15, 2024

This will run the notability check earlier, so that maintainers don't spend time fixing other syntax issues that are only checked later by test bot.

  • Have you followed the guidelines for contributing?
  • Have you ensured that your commits follow the commit style guide?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

This will run the notability check earlier, so that maintainers don't spend time fixing other syntax issues that are only checked later by test bot.
@github-actions github-actions bot added workflows PR modifies GitHub Actions workflow files automerge-skip `brew pr-automerge` will skip this pull request labels Jul 15, 2024
@iMichka
Copy link
Member Author

iMichka commented Jul 15, 2024

This is an early draft:

  • Not sure if this is the best place to add this, as we are starting to add unrelated stuff to the formulae_detect step. But we can still split this up in a second pass?
  • Not sure if we want to tweak test bot to not redo the audit again later?

@stefanb
Copy link
Member

stefanb commented Jul 16, 2024

Shouldn't it be also removed from later tests?

@MikeMcQuaid
Copy link
Member

  • Not sure if this is the best place to add this, as we are starting to add unrelated stuff to the formulae_detect step. But we can still split this up in a second pass?

Instead I would add this to the tap_syntax step which is run earlier and is quicker to run. Think you'll need an --online for this, too, and probably should put it in brew test-bot instead.

Copy link
Contributor

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@github-actions github-actions bot added the stale No recent activity label Aug 15, 2024
@github-actions github-actions bot closed this Aug 22, 2024
@github-actions github-actions bot deleted the audit branch August 22, 2024 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge-skip `brew pr-automerge` will skip this pull request stale No recent activity workflows PR modifies GitHub Actions workflow files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants