-
-
Notifications
You must be signed in to change notification settings - Fork 10.9k
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
trae-cn 1.0.8055 (new cask) #203613
trae-cn 1.0.8055 (new cask) #203613
Conversation
Looks fine to me, but the token might be worth discussing. We have two existing casks with Chinese variants using different naming schemes: Since we usually use the |
Yeah I think I would agree here. We should be using @ for alpha, beta, etc. not different languages. This is also probably worth a look through Casks and finding discrepancies and correcting. |
The token reference doesn't cleanly apply, because not every geographical variant will have a different app name. In some cases it makes could make sense to use I lean towards using the |
I go through some of the casks How
|
Hi @khipp @krehel @bevanjkay , can you review it again? THX~ |
Whichever direction we decide for the token, we should update the docs, and make sure the other casks are consistent here. I still lean toward using the |
I will let you make the call but would like to offer another perspective. Using
|
Makes sense. I was thinking about this earlier. |
That should be possible, but it would be one of our more complex casks since we would also have to substitute parts of the artifact name and zap paths. |
It's probably not worth the added complexity, we can leave it as is |
Important: Do not tick a checkbox if you haven’t performed its action. Honesty is indispensable for a smooth review process.
In the following questions
<cask>
is the token of the cask you're submitting.After making any changes to a cask, existing or new, verify:
brew audit --cask --online <cask>
is error-free.brew style --fix <cask>
reports no offenses.Additionally, if adding a new cask:
brew audit --cask --new <cask>
worked successfully.HOMEBREW_NO_INSTALL_FROM_API=1 brew install --cask <cask>
worked successfully.brew uninstall --cask <cask>
worked successfully.Trae CN is a special edition of Trae:
PS: It's Trae's official edition, not some wild custom edition.