493 Commits

Author SHA1 Message Date
Markus Reiter
490e503b1b Replace Homebrew.args with Context. 2020-08-06 16:13:14 +02:00
Rylan Polster
3b1c8ce37e
Merge pull request #8161 from Rylan12/add-update-python-resources-blocklist
update-python-resources: add blocklist
2020-08-03 17:08:56 -04:00
Rylan Polster
9fb3a80e38 Add cloudformation-cli to blocklist 2020-08-02 11:30:10 -04:00
Rylan Polster
1f5479798c Add ansible and ansible@2.8 to blocklist 2020-07-31 20:59:49 -04:00
Rylan Polster
57cac49559 Use formula.full_name for blocklist 2020-07-31 20:59:22 -04:00
Rylan Polster
075ae9eb1e update-python-resources: add blocklist 2020-07-31 17:01:30 -04:00
Rylan Polster
9dd52a83e4 Reword comment above get_pypi_info 2020-07-31 15:12:41 -04:00
Rylan Polster
b08d1a28ad update-python-resources: handle no sdist for package 2020-07-31 14:21:44 -04:00
Bo Anderson
24249e1f3e
Merge pull request #8155 from Bo98/pipgrip-defer
utils/pypi: don't require pipgrip unless we are using it
2020-07-31 17:49:11 +01:00
Rylan Polster
53a27185a6
Merge pull request #8146 from Rylan12/update-python-resources-fix
update-python-resources: add --no-cache-dir flag
2020-07-31 12:18:00 -04:00
Bo Anderson
92cee70cbd utils/pypi: don't require pipgrip unless we are using it 2020-07-31 16:56:21 +01:00
Rylan Polster
ec7be559f4 Refactor package name and url logic to PyPI module 2020-07-31 10:10:07 -04:00
Rylan Polster
03cf0e0180 update-python-resources: add --no-cache-dir flag 2020-07-30 14:26:27 -04:00
Rylan Polster
2d086013f9 dev-cmd: add update-python-resources command 2020-07-29 09:50:16 -04:00
Mike McQuaid
a85eb3fa49
Revert "curl: alert on 301 redirect" 2020-07-28 10:38:27 +01:00
Issy Long
fdbb2ac16a
Merge pull request #7985 from vidusheeamoli/utils-inreplace-to-true
sorbet: set utils/inreplace.rb to true
2020-07-26 10:37:30 +01:00
vidusheeamoli
fd382d2ecd srb: set utils/inreplace.rb to true and refactor
- Sorbet gives preference to class methods over methods defined in
  included modules, hence Sorbet was unavailable to resolve the
definition of the gsub! method.
- The gsub! method in StringInreplaceExtension conflicts with the definition in String.
- This PR refactors the call to the gsub! method so that a custom object
  is exposed instead of a string.
2020-07-26 12:43:31 +05:30
Markus Reiter
b50eea6849
Merge pull request #8068 from reitermarkus/cli-parser
Refactor usages of global `Homebrew.args`.
2020-07-25 21:01:39 +02:00
Markus Reiter
6c050492ee Pass args to Analytics instead of using global args. 2020-07-25 03:57:43 +02:00
Dustin Rodrigues
c4d11374c8 curl: alert on permanent redirect 2020-07-24 10:50:13 -04:00
Dustin Rodrigues
ef540d0365 curl: ensure final_url is populated with the final URL requested 2020-07-22 17:41:55 -04:00
Dawid Dziurla
b4efb2c258
notability: fix variable names 2020-07-22 14:21:06 +02:00
Sean Molenaar
dab18076fa
audit: error on archived repos 2020-07-22 11:57:50 +02:00
Dawid Dziurla
d579725f7c
Merge pull request #7954 from dawidd6/workflow-dispatch-event
utils/github: add workflow_dispatch_event method
2020-07-16 19:08:12 +02:00
vidusheeamoli
9799f32506 sorbet: set utils/user.rb to true 2020-07-11 01:04:22 +05:30
Dawid Dziurla
de4f25c13d
utils/github: add workflow_dispatch_event method 2020-07-09 12:50:09 +02:00
Issy Long
2c3b2f68cd
Revert #7933 ("sorbet: set utils/git.rb to true")
- My refactoring of #7933 went wrong in that the tests passed for `brew
  extract` and my manual testing, but both forgot about third-party taps
  exist, so that functionality broke as follows (courtesy of Misty).

  Before:

  ```
  # Git.last_revision_commit_of_files("/usr/local/Homebrew", ["LICENSE.txt", "README.md"])
  => ["ac0665d", ["README.md"]]
  ```

  After:

  ```
  # Git.last_revision_commit_of_files("/usr/local/Homebrew", ["LICENSE.txt", "README.md"])
  => [nil, []]
  ```

- While we think about how to do splats in Sorbet, revert this so
  that users are able to `brew extract` from third party taps again.

- A TODO for later in a separate PR is to write a test for `brew
  extract` that tests the behaviour of third-party taps.

---

- Reverted this manually because the GitHub UI couldn't do it.
- Arguably I didn't need to remove the RBI file, but it's easier to have
  everything gone for now and then revert this revert in future once we
  have a strategy for dealing with splats in Sorbet than have
  inconsistency.
2020-07-09 08:18:38 +01:00
vidusheeamoli
76a2e69990 sorbet: set utils/git.rb to true
(cherry picked from commit 44a04adb1de430066af73a65766eb644c0f5bd41)
2020-07-08 22:34:26 +05:30
Mike McQuaid
15a2abd9df
style: re-enable shellcheck.
Ensure that `shellcheck` is run by `brew style` again and fix the
various failures.
2020-07-07 11:53:55 +01:00
Jonathan Chang
b909fdd30f github: refactor GraphQL APIs 2020-07-05 15:31:29 +10:00
Bo Anderson
997ba42a4b utils/curl: support disabling partial downloads in curl_download 2020-07-02 18:58:32 +01:00
Jonathan Chang
d16e699ed7 github: need read-only email scope to see emails 2020-07-01 22:25:53 +10:00
Jonathan Chang
b57290b23f
Merge pull request #7844 from jonchang/signoff-approving-reviews
pr-pull: add commit message trailers for pull requests with approved reviews
2020-07-01 19:18:27 +10:00
lionellloh
57e7e7d60e small fix: res -> response 2020-06-30 23:23:19 +08:00
Lionell
fb4db63e3b Merge branch 'master' into license 2020-06-30 22:25:17 +08:00
lionellloh
96d60c271a Merge remote-tracking branch 'mlh/license' into license 2020-06-30 22:12:03 +08:00
Lionell Loh Jian An
5b3530b23d
Apply suggestions from code review
Code review changes

Co-authored-by: Mike McQuaid <mike@mikemcquaid.com>
2020-06-30 22:11:36 +08:00
lionellloh
c50f7b8951 Revert "curl: modified curl to allow optional print_stdout"
This reverts commit 02bc870e915f88145761b11d2065d8f2964c50ba.
2020-06-30 21:58:40 +08:00
lionellloh
02bc870e91 curl: modified curl to allow optional print_stdout 2020-06-30 01:13:02 +08:00
Jonathan Chang
9cab9b7f39
pr-pull: handle empty string cases
Co-authored-by: Mike McQuaid <mike@mikemcquaid.com>
2020-06-29 18:55:47 +10:00
Mike McQuaid
61b4e7c414
Merge pull request #7745 from dtrodrigues/bump-pr-dupes
bump-formula-pr: search for closed dupe PRs
2020-06-29 08:39:30 +01:00
Jonathan Chang
90309e5f42 github: fetch approved reviews for a pull request 2020-06-29 12:53:03 +10:00
Dustin Rodrigues
f684a59fa5 bump-formula-pr: search for closed dupe PRs 2020-06-27 12:50:43 -04:00
Lionell
57fd1fdc7e Merge branch 'master' into license 2020-06-25 13:42:27 +08:00
Mike McQuaid
07ea18e2d2
ruby.sh: don't print recoverable error.
This prints "error" even when we can install a portable ruby.
2020-06-23 13:13:31 +01:00
Lionell Loh Jian An
eca528cccc
util github: remove nil as that is already expected behaviour
Co-authored-by: Mike McQuaid <mike@mikemcquaid.com>
2020-06-19 03:38:36 +08:00
Lionell
610dfe0fc2 audit: fixed with PR comments. Refactored get_repo_license api call and audit_license only new_formula 2020-06-18 00:44:33 +08:00
Mike McQuaid
854e96c7c5
Merge pull request #7686 from maxim-belkin/test_ruby_update
utils/ruby.sh: update `test-ruby` (and rename to test_ruby)
2020-06-12 09:13:15 +01:00
Bo Anderson
87931e1c03 utils/ruby.sh: set TERMINFO_DIRS to bundled terminfo 2020-06-10 14:36:18 +01:00
Maxim Belkin
23d24dfed7
utils/ruby.sh: rely on test_ruby's exit code
Instead of printing "true" or "false", test_ruby now returns 1 when
tested Ruby is not usable and 0 if it is.
2020-06-10 07:56:37 -05:00