| Age | Commit message (Collapse) | Author |
|
|
|
|
|
|
|
Also, flag those that we never want to be in a manpage.
|
|
`e.tap` calls the tap method which wants a block on
`FormulaUnavailableError` so the previous approach is :-1:.
|
|
Handle the case where a `FormulaUnavailableError` does not contain a tap
reference so fall back to `CoreTap` in this case.
|
|
This will be useful in debugging (and recovering from) the situation
where sometimes formulae can't be found when specifying multiple on the
command-line.
|
|
Always run `git clean -ffdx` on HOMEBREW_REPOSITORY. Also, ensure that
if we can't find a formula that's logged.
|
|
There's edge-cases which is breaking tap CI.
|
|
Otherwise weird things happen and formulae can't be found.
|
|
This is set by Jenkins to indicate a downstream bottle upload job needs
to use `brew bottle --keep-old`.
|
|
test-bot: cleanup all repositories.
|
|
bump-formula-pr: audit formula before opening PR
note that it was made opt-in not opt-out
|
|
The test bot currently only focuses on doing cleanup on the current tap
and the HOMEBREW_REPOSITORY. This should be extended to cover all
repositories as e.g. a syntax error in homebrew/core can break the CI
for all the other taps.
|
|
|
|
If there's formulae changes or the specified argument is a formula then
don't waste time running e.g. `brew tests`.
|
|
|
|
Also report Cask coverage.
|
|
This allows changed formulae in taps to be tested in the proper order.
See #738 for more details.
|
|
|
|
|
|
Flips HOMEBREW_RUBY_MACHO to HOMEBREW_NO_RUBY_MACHO.
|
|
bump-formula-pr: return to starting branch
|
|
After branching, bumping, pushing, and pr-ing; return to whatever branch
was originally checked out.
In most cases, I'd imagine users to want to continue receiving tap
updates from master. However, after using bump-formula-pr, the tap in
which the formula was bumped is left on the working branch that was
doing the bumping and pull-request. After opening the PR, we should
return to whatever branch the user originally had checked out – most
likely master. (But git allows us to just say "previous branch" by using
`-`)
|
|
Without `--no-track`, some git setups may automatically set
`origin/master` as the tracked upstream for the newly created branch.
This upstream is what hub defaults as --head when opening PRs. By not
allowing git to set `origin/master` as upstream, hub can then use the
proper --head for the PR.
Since hub still needs to know what --head is intended to be, we can set
the branch's upstream when pushing: with the --set-upstream option.
Fixes #755
|
|
|
|
|
|
|
|
Assume Ruby 2
|
|
|
|
- ignore Cask's files in `readall` (for now, there's an intentional
syntax error that will need fixed)
- run Cask's tests if they exist
- don't check Cask's files in coverage reports (for now)
|
|
- Inconsistent or unneeded indentation
- Missing or superfluous empty lines
- Missing or wrongly formatted arguments in command summary
- Missing punctuation
|
|
Loosen this a bit; we don't necessarily expect the end commit is the one
we're looking for, just that it has changed from the start commit (i.e.
some sort of update has occurred).
Addresses some false negatives on `master` branch merges that weren't
present on the PR commits.
|
|
This would have caught the bug fixed in 985c67.
|
|
This can just live in `brew.sh` and then it doesn’t need repeated in
all the other places.
|
|
|
|
This reverts commit 24f7e671317dfe22f1d8e10426db2e9074674bc9.
|
|
|
|
|
|
When running on Travis CI, both the Linux and macOS build will send a
coverage report, causing them to be merged by Coveralls. This results
in inferior coverage due to the early stage of the Linux-specific tests
and is probably not what we want. Make sure we only send a report for
macOS (assuming we stick with a single macOS build in `.travis.yml`).
|
|
`any?` is not the opposite of `empty?`. Besides the case that
`[false, nil].any?` will return false, `any?`(O(n)) has much worse
performance than `empty?`(O(1)).
|
|
Rather than nudge people to run `--strict` and then ignore some of the
results sometimes (e.g. GitHub repository notability) instead add a
dedicated `--new-formula` option that implies this is a one-time
advisory check.
|
|
|
|
|
|
|
|
|
|
|
|
Make sure to call `brew tests` only once with `--coverage` to avoid
expensive multiple runs and to prevent later runs from overwriting
previously sent results to Coveralls. (The previous setup overwrote the
results from a regular run with results from the `--generic` run.)
The `--no-compat` variant without any other options specified seemed to
be the most appropriate for a coverage report.
Closes #546.
Signed-off-by: Martin Afanasjew <martin@afanasjew.de>
|
|
|
|
|