-
Notifications
You must be signed in to change notification settings - Fork 332
Updated name in ascii banner #1654
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
|
I am not sure we can say Apache yet without incubating? Probably not a big deal. |
|
We can for the banner/logo |
snazy
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we maybe let the whole community decide?
Maybe with 2 or 3 alternatives, where everybody has an equal +1/0/-1 choice?
quarkus/admin/src/main/resources/org/apache/polaris/admintool/banner.txt
Outdated
Show resolved
Hide resolved
dimas-b
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @scottteal ! I like the new ascii art 🙂
| @@@@ | ||
| @@@ | ||
| @@ | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: remove the trailing new line. Quarkus adds Powered by Quarkus 3.21.4 (right-justified) automatically. I think it'll look better without the extra empty line.
|
Having alternatives SGTM. I'll open one from my side presently. |
Contributes to apache#1653 Alternative to apache#1654
|
Alternative : #1655 Shall we wait a couple of days for other entries? |
Sure no problem as long we get a few :) I would never want to squash someone's artistic drive. How about we wait till 5/28 and link all alternatives in the parent issue #1653 and we can do approval voting with emojis. I would end the vote on Friday 5/30 and merge the winner. I don't think it's super serious since we could even change the banner every release if we wanted to. That would be kinda fun as well |
dimas-b
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving this proposal based on votes in #1653 :

Issue: #1653
Dev List: https://lists.apache.org/thread/bdv588pdn361852oj1cm81govl75m51h
Updating the ASCII banner art to reflect the project's name "Apache Polaris"