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 Dbvisualizer.app v9.1.11 #6668

Closed
wants to merge 2 commits into from

Conversation

junior-ales
Copy link
Contributor

Adding a cask for DbVisualizer universal database tool. It was made for developers, DBAs and analysts. It is the perfect solution since the same tool can be used on all major operating systems accessing a wide range of databases.

@rolandwalker
Copy link
Contributor

Hi! Thanks!

Since the app bundle here is a GUI installer rather than the app itself, app isn't the right stanza. Instead of app 'DbVisualizer Installer.app', we should say

caveats do
  manual_installer('DbVisualizer Installer.app')
end

to alert users that they need to take an additional step.

After #6660, it will be possible to do that without needing a caveats block.

@junior-ales
Copy link
Contributor Author

Thanks for the call out. Stanza updated.

@vitorgalvao
Copy link
Contributor

Thank you for the contribution. It will be merged in a different pull request due to some simple recommended alterations. Your contribution is still included (and still credited to you), with the appropriate modifications, when applicable.

Changes:

@Homebrew Homebrew locked and limited conversation to collaborators May 8, 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.

3 participants