Skip to content

Comments

Fix uninitialized constant Set error#249098

Open
danielbayley wants to merge 1 commit intoHomebrew:mainfrom
danielbayley:dev-list-running-app-ids
Open

Fix uninitialized constant Set error#249098
danielbayley wants to merge 1 commit intoHomebrew:mainfrom
danielbayley:dev-list-running-app-ids

Conversation

@danielbayley
Copy link
Contributor

Important: Do not tick a checkbox if you haven’t performed its action. Honesty is indispensable for a smooth review process.

In the following questions <cask> is the token of the cask you're submitting.

After making any changes to a cask, existing or new, verify:

Additionally, if adding a new cask:

  • Named the cask according to the token reference.
  • Checked the cask was not already refused (add your cask's name to the end of the search field).
  • brew audit --cask --new <cask> worked successfully.
  • HOMEBREW_NO_INSTALL_FROM_API=1 brew install --cask <cask> worked successfully.
  • brew uninstall --cask <cask> worked successfully.

If AI was used to generate or assist with generating the PR:

  • I used AI to generate or assist with generating this PR. Please specify below how you used AI to help you.
  • I have personally reviewed, tested and verified all changes/additions, including zap stanza paths.

@SMillerDev
Copy link
Member

CI disagrees that this is needed. Are you using the required ruby version?

@danielbayley
Copy link
Contributor Author

CI disagrees that this is needed. Are you using the required ruby version?

@SMillerDev Ah, I’m stuck on Sequoia (15.7.3, with /usr/bin/ruby 2.6.10) at the moment due to hardware issues… Is it worth removing this one line to make contributing more difficult for users on slightly older OS versions?

@SMillerDev
Copy link
Member

It should use portable ruby though. Not the system one

@danielbayley
Copy link
Contributor Author

It should use portable ruby though. Not the system one

This is a bug then?

@bevanjkay
Copy link
Member

What if you first run brew sh?

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.

4 participants