-
-
Notifications
You must be signed in to change notification settings - Fork 12.9k
postgrestools 0.3.1 (new formula) #218049
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
Conversation
Thanks for contributing to Homebrew! 🎉 It looks like you're having trouble with a CI failure. See our contribution guide for help. You may be most interested in the section on dealing with CI failures. You can find the CI logs in the Checks tab of your pull request. |
6aa4404
to
dbba362
Compare
If it is not a requirement of the build system to build the project with |
dbba362
to
a7a39c1
Compare
a7a39c1
to
641047a
Compare
cd23d34
to
497f517
Compare
497f517
to
2969529
Compare
I switched to clone because we need the submodules for |
I request that this formula not be called "postgrestools". That's too generic and not useful. It would make more sense and be more recognizable to use "postgres-language-server". |
The website is called pgtools though. Is it not confusing to use a different name than the public website? |
Well, the website is "pgtools", the source code repo is "postgres-language-server", and the proposed formula is "postgrestools". Which one is it? They're not doing themselves any favors in their branding. In any case, I'm a developer from the PostgreSQL project. I'm pretty confident that no one in that community would think of this piece of software when they think of "postgres tools". So I think this would be inappropriate name squatting and prone to mislead users. Maybe something like "supabase-pgtools" would be clearer. |
thanks for your feedback! I totally get the conflict of interest here. for transparency: the project is evolving from just a language server to what I as an outsider would call a language toolchain that includes a linter and eventually a formatter among other things. Still, I agree that "Postgres Tools" can be misleading for many. We also had "Postgres Language Tools" in mind, but found the name to be too long. What do you think about these?
Just pick one and I will go for it :) We would like to keep Supabase out of the name though. The project is supposed to be for anyone building with Postgres, not just Supabase users. They are just supporting us with resources. Sorry for the forth and back! |
Personally, I think postgres-language-server has good brand recognition. There were several Hacker News etc. stories about it, and it's a project I would actually try to go out and find. The other stuff, evolving it into a box of language tools, I think that's either (a) not that important, the current name is good enough, or (b) a problem for the future, you can always rename it, or (c) then you think of a completely new and unique name, like superelephant or pglinguist or some artificial word. |
It makes sense to me to keep this as |
@petere - thanks for weighing in here. We'd love to keep it simple, and at the same time make it clear that it's a contribution for the pg community - not specific to Supabase. (In that vein we're happy to relocate the repo it into it's own GitHub org, accept other contributors, rename it, donate it - generally whatever is required to make that obvious.)
@psteinroe - on Peter's point here, my preference is also @petere if you agree with |
Currently,
In the future,
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>
, where<formula>
is the name of the formula you're submitting?brew test <formula>
, where<formula>
is the name of the formula you're submitting?brew audit --strict <formula>
(after doingHOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>
)? If this is a new formula, does it passbrew audit --new <formula>
?