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

Update readme with Homebrew installation support #66

Merged
merged 3 commits into from
Jun 22, 2020
Merged

Update readme with Homebrew installation support #66

merged 3 commits into from
Jun 22, 2020

Conversation

Kaspik
Copy link
Member

@Kaspik Kaspik commented Jun 22, 2020

Improves #58

  • remove build info - this was in Sequel-Pro so people could build master project
  • add Homebrew info
  • update styling

Screen Shot 2020-06-22 at 22 34 27

@Jason-Morcos
Copy link
Member

Love it. Would it be worth keeping the build info under the contributing section to help newcomers know how to get started?

@Kaspik
Copy link
Member Author

Kaspik commented Jun 22, 2020

I'm not sure about that. People that want to contribute to such a complicated (bug in this app can cause issues for millions of users if it breaks prod. database) project should know how to clone a repo and open a XCode project, what do you think?

@Jason-Morcos
Copy link
Member

I'm not sure about that. People that want to contribute to such a complicated (bug in this app can cause issues for millions of users if it breaks prod. database) project should know how to clone a repo and open a XCode project, what do you think?

Hmmm they should know, but on the other hand we never want to have a barrier to entry for someone to start playing with something, if only locally. I could go either way! Can let others chime in.

@juniorb2ss
Copy link

Is it expected to close the issue #58?

The way to install using mas is great, however, it also requires another library. The interesting thing is to keep the issue open until we can install it directly through the cask.

What you think?

@Kaspik
Copy link
Member Author

Kaspik commented Jun 22, 2020

Not close, we will leave it open. :)

@juniorb2ss
Copy link

Thank you guys. I'm very happy to see this project still in progress. Nice job.

@Kaspik Kaspik merged commit ec1f412 into master Jun 22, 2020
@Kaspik Kaspik deleted the homebrew branch June 22, 2020 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants