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

ci: add workflow for testing formulae #5

Merged
merged 18 commits into from
Jun 16, 2023
Merged

Conversation

shamilsan
Copy link
Contributor

@shamilsan shamilsan commented Jun 16, 2023

  • Add a CI workflow to test formulae.

Note: Now in fails on Linux due to #4

@shamilsan shamilsan marked this pull request as ready for review June 16, 2023 07:39
@shamilsan shamilsan requested a review from clearloop June 16, 2023 07:39
@shamilsan shamilsan marked this pull request as draft June 16, 2023 07:42
@shamilsan shamilsan removed the request for review from clearloop June 16, 2023 07:42
@shamilsan shamilsan marked this pull request as ready for review June 16, 2023 07:50
@shamilsan shamilsan requested a review from clearloop June 16, 2023 07:50
Copy link
Contributor

@clearloop clearloop left a comment

Choose a reason for hiding this comment

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

lgtm!

  • since we have checksum, do we really need version checks?
  • should we support linux version of brew?
  • scheduled or not?

@clearloop
Copy link
Contributor

clearloop commented Jun 16, 2023

I'm worrying about once we have tons of versions of gear, just figured out a trick doing these,

brew tap-info gear-tech/gear --json
[
  {
    "name": "gear-tech/gear",
    "user": "gear-tech",
    "repo": "gear",
    "path": "/opt/homebrew/Library/Taps/gear-tech/homebrew-gear",
    "installed": true,
    "official": false,
    "formula_names": [
      "gear-tech/gear/[email protected]",
      "gear-tech/gear/[email protected]",
      "gear-tech/gear/[email protected]",
      "gear-tech/gear/vara@stable",
      "gear-tech/gear/getgear",
      "gear-tech/gear/[email protected]",
      "gear-tech/gear/[email protected]",
      "gear-tech/gear/gear",
      "gear-tech/gear/vara",
      "gear-tech/gear/[email protected]",
      "gear-tech/gear/gear@nightly",
      "gear-tech/gear/[email protected]",
      "gear-tech/gear/[email protected]"
    ],
    "formula_files": [
      "/opt/homebrew/Library/Taps/gear-tech/homebrew-gear/Formula/[email protected]",
      "/opt/homebrew/Library/Taps/gear-tech/homebrew-gear/Formula/[email protected]",
      "/opt/homebrew/Library/Taps/gear-tech/homebrew-gear/Formula/[email protected]",
      "/opt/homebrew/Library/Taps/gear-tech/homebrew-gear/Formula/[email protected]",
      "/opt/homebrew/Library/Taps/gear-tech/homebrew-gear/Formula/getgear.rb",
      "/opt/homebrew/Library/Taps/gear-tech/homebrew-gear/Formula/[email protected]",
      "/opt/homebrew/Library/Taps/gear-tech/homebrew-gear/Formula/[email protected]",
      "/opt/homebrew/Library/Taps/gear-tech/homebrew-gear/Formula/gear.rb",
      "/opt/homebrew/Library/Taps/gear-tech/homebrew-gear/Formula/vara.rb",
      "/opt/homebrew/Library/Taps/gear-tech/homebrew-gear/Formula/[email protected]",
      "/opt/homebrew/Library/Taps/gear-tech/homebrew-gear/Formula/[email protected]",
      "/opt/homebrew/Library/Taps/gear-tech/homebrew-gear/Formula/[email protected]",
      "/opt/homebrew/Library/Taps/gear-tech/homebrew-gear/Formula/[email protected]"
    ],
    "cask_tokens": [

    ],
    "cask_files": [

    ],
    "command_files": [
      "/opt/homebrew/Library/Taps/gear-tech/homebrew-gear/cmd/glinks.rb"
    ],
    "remote": "[email protected]:gear-tech/homebrew-gear.git",
    "custom_remote": -1,
    "private": true
  }
]

we can get all of the formulas from brew tap-info gear-tech/gear --json with json format, then, with some scripts mb jq, we get all of the versions and testing the downloading

@shamilsan
Copy link
Contributor Author

I'm worrying about once we have tons of versions of gear, just figured out a trick doing these,

I think we are to abandon supporting the old versions (even maybe with the formulae removing). Using the matrix allows running checks in parallel.

@shamilsan shamilsan merged commit 14849b1 into master Jun 16, 2023
@shamilsan shamilsan deleted the sy-add-ci-workflows branch June 16, 2023 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants