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

Add Noti 0.1.5 #25429

Merged
merged 2 commits into from
Oct 12, 2016
Merged

Add Noti 0.1.5 #25429

merged 2 commits into from
Oct 12, 2016

Conversation

jariz
Copy link
Contributor

@jariz jariz commented Oct 12, 2016

Noti is a pushbullet client for macOS. It allows you to see and act upon your Android notifications from macOS.
Homepage: https://noti.center/

  • brew cask audit --download {{cask_file}} is error-free.
  • brew cask style --fix {{cask_file}} reports no offenses.
  • The commit message includes the cask’s name and version.
  • Named the cask according to the token reference.
  • brew cask install {{cask_file}} worked successfully.
  • brew cask uninstall {{cask_file}} worked successfully.
  • Checked there are no open pull requests for the same cask.
  • Checked that the cask was not already refused in closed issues.


# github.com/jariz/Noti was verified as official when first introduced to the cask
url "https://github.com/jariz/Noti/releases/download/#{version}/Noti.dmg"
appcast 'https://rawgit.com/jariz/Noti-appcast/master/appcast.xml',
Copy link
Member

Choose a reason for hiding this comment

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

Use https://github.com/jariz/Noti/releases.atom.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thought it'd make more sense to use the sparkle url because the app uses it as well, but will do 👍

@jariz
Copy link
Contributor Author

jariz commented Oct 12, 2016

Regarding the build check, does it really matter that a brew formula already exists? I thought cask & brew used a separate repo?
Eitherway, the one in brew is not related to my project. I can possibly rename it, but would rather not.

@vitorgalvao
Copy link
Member

vitorgalvao commented Oct 12, 2016

Regarding the build check, does it really matter that a brew formula already exists?

Only if the apps are related, which is not the case here. It’s a warning, not a failure.

@vitorgalvao vitorgalvao merged commit d713364 into Homebrew:master Oct 12, 2016
@jariz jariz deleted the noti branch October 12, 2016 12:50
@Homebrew Homebrew locked and limited conversation to collaborators May 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants