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

Use File instead of IO for file operations #105367

Merged
merged 3 commits into from
May 11, 2021
Merged

Use File instead of IO for file operations #105367

merged 3 commits into from
May 11, 2021

Conversation

Rylan12
Copy link
Member

@Rylan12 Rylan12 commented May 8, 2021

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

After making all changes to a cask, verify:

See Homebrew/brew#11140

Replace IO.write and IO.read with File.write and File.read, respectively, in j, minecraft-server, and openzfs.

Note that I was unable to test this for openzfs because it depends on certain macOS versions that I don't have access to (i.e. doesn't work on Big Sur).

@Rylan12
Copy link
Member Author

Rylan12 commented May 8, 2021

CC @samford because I'm not sure about some of these livecheck blocks

vitorgalvao
vitorgalvao previously approved these changes May 8, 2021
Casks/j.rb Outdated Show resolved Hide resolved
Casks/minecraft-server.rb Outdated Show resolved Hide resolved
Casks/openzfs.rb Outdated Show resolved Hide resolved
Bo98
Bo98 previously approved these changes May 10, 2021
@Bo98
Copy link
Member

Bo98 commented May 10, 2021

Thanks!

@Rylan12 Rylan12 merged commit f47ea35 into Homebrew:master May 11, 2021
@Rylan12 Rylan12 deleted the file-instead-of-io branch May 11, 2021 13:05
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 11, 2021
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants