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

writerside 2023.2,232.10165 (new cask) #157813

Merged
merged 1 commit into from
Oct 18, 2023
Merged

Conversation

dahlia
Copy link
Contributor

@dahlia dahlia commented Oct 17, 2023

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.
  • Checked the cask is submitted to the correct repo.
  • brew audit --new-cask <cask> worked successfully.
  • brew install --cask <cask> worked successfully.
  • brew uninstall --cask <cask> worked successfully.

Comment on lines 27 to 34
uninstall_postflight do
ENV["PATH"].split(File::PATH_SEPARATOR).map { |path| File.join(path, "rover") }.each do |path|
if File.readable?(path) &&
File.readlines(path).grep(/# see com.intellij.idea.SocketLock for the server side of this interface/).any?
File.delete(path)
end
end
end
Copy link
Contributor

@miccal miccal Oct 17, 2023

Choose a reason for hiding this comment

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

This is copied from the rustrover Cask, and I assume will need to be updated.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It turns out that Writerside's command line launcher is named writerside, so I fixed it with that name.

@miccal miccal added the awaiting user reply Issue needs response from a user. label Oct 17, 2023
@dahlia dahlia requested a review from miccal October 18, 2023 04:20
@miccal miccal removed the awaiting user reply Issue needs response from a user. label Oct 18, 2023
@miccal
Copy link
Contributor

miccal commented Oct 18, 2023

Thank you @dahlia.

@miccal miccal merged commit ceb3b61 into Homebrew:master Oct 18, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 22, 2023
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.

3 participants