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

singlecrystal: 5.0.0 (new cask) #172225

Merged
merged 3 commits into from Apr 28, 2024
Merged

Conversation

singularitti
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.
  • Checked the cask is submitted to the correct repo.
  • 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.

@singularitti
Copy link
Contributor Author

  1. I was unable to find a link like https://crystalmaker.com/support/updates/VersionData-SGM.xml, please help me with the livecheck.
  2. The brew style --fix will throw an error because desc contains "Single-crystal", but it is a scientific term, and I don't think that duplication matters.

Copy link
Member

@samford samford left a comment

Choose a reason for hiding this comment

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

  1. I was unable to find a link like https://crystalmaker.com/support/updates/VersionData-SGM.xml, please help me with the livecheck.

Looks like the URL is https://crystalmaker.com/support/updates/VersionData-SCM.xml. I pushed a commit to add a livecheck block, using the approach in https://github.com/Homebrew/homebrew-cask/pull/172222/files#r1581568335.

Past the livecheck block, I'll leave the overall cask review to our cask maintainers.

@singularitti
Copy link
Contributor Author

Ah, it was my oversight. Thank you so much @samford!

@razvanazamfirei razvanazamfirei added the ready to merge PR can be merged once CI is green label Apr 27, 2024
@singularitti singularitti changed the title singlecrystal: 5.0.0 (new cask) singlecrystal: 5.0.0 (new cask) Apr 27, 2024
@bevanjkay bevanjkay merged commit 6134641 into Homebrew:master Apr 28, 2024
12 checks passed
@singularitti singularitti deleted the singlecrystal branch April 28, 2024 23:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new cask ready to merge PR can be merged once CI is green
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants