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

alot of reworking on both the portuguese (eu and br), spanish and english pages #1005

Closed
wants to merge 0 commits into from

Conversation

65hz
Copy link
Contributor

@65hz 65hz commented Dec 22, 2023

What did I do?

I made some slightly significant changes to the 4 pages above. You can see this in the caskinstall, pkginstaller, and githubrelease sections as the French version of githubrelease sounded a lot better than the English, and the Portuguese (EU & BR) and Spanish pages (which I based off of the English version), also better conjugation in some areas.


I also added a little section in the index.html called submit, right now I only made the English version but I might consider adding it to pt and es.


I think this really should be checked a lot more than my previous commits in case they sound too overenthusiastic, or poorly worded, or both, but I think they all sound pretty good.

caskcreate: Making a cask is as simple as creating a formula.
install:
install: Install Homebrew
paste: Paste that in a macOS Terminal or Linux shell prompt.
what: The script explains what it will do and then pauses before it does it. Read about other <a href="https://docs.brew.sh/Installation">installation options</a>.
pkginstaller: If you're on macOS, try our new <code>.pkg</code> installer.
githubrelease: Download it from <a href="https://github.com/Homebrew/brew/releases/latest">Homebrew's latest GitHub release</a>.
githubrelease: Download it from the <a href="https://github.com/Homebrew/brew/releases/latest">"GitHub releases"</a> page.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But this doesn't link there, it links to the latest release

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess I can change up the link to make it link to the releases page or change up the text to convey it is the latest release, but from what I have seen, the French page also does this? I might try reversing the change but I don't know.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the original text was better since it didn't require knowing about how releases are displayed in GitHub

prefix: Homebrew won’t install files outside its prefix, but you can place a Homebrew installation wherever you want.
createpackages: You can trivially create your own Homebrew packages.
hack: After all, it’s just Git and Ruby underneath, you can easily hack away with the knowledge, revert your modifications, and merge upstream updates.
submit: And if you want, you can submit your package to us <a href="https://github.com/Homebrew/homebrew-core/pulls">here</a>!
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should link to the docs because casks exist too

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah true, I'll get to doing that.

Copy link
Contributor Author

@65hz 65hz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

made the changes, it should be good now

Copy link
Member

@EricFromCanada EricFromCanada left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not a fan of the en.yml changes, especially since they'll render all the other translations out of date. Any changes to the English should either be extremely minor corrections or part of a coordinated rewrite.

@65hz
Copy link
Contributor Author

65hz commented Dec 23, 2023

For some bits, I did attempt to translate it over (like the "you dont need to use the mouse" bit of caskinstall) but if its only that and some other heavily modified areas I am happy to change that over

@65hz
Copy link
Contributor Author

65hz commented Dec 23, 2023

For some bits, I did attempt to translate it over (like the "you dont need to use the mouse" bit of caskinstall) but if its only that and some other heavily modified areas I am happy to change that over

*translate it over to the languages I mean, forgot to give more context

Copy link
Member

@SMillerDev SMillerDev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you split the submit change from the rewrite?

@65hz
Copy link
Contributor Author

65hz commented Dec 23, 2023

Can you split the submit change from the rewrite?

Do you mean make it a different branch?

@SMillerDev
Copy link
Member

Make it a different pull request, yes

@65hz
Copy link
Contributor Author

65hz commented Dec 24, 2023

@SMillerDev I have made it a different branch.
@EricFromCanada Reversed the major English changes.

Copy link
Member

@MikeMcQuaid MikeMcQuaid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR! Let's strip this into at least two instead, please.

@@ -1,5 +1,5 @@
pt:
locale_name: Português
locale_name: Português (europeu)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
locale_name: Português (europeu)
locale_name: Português

is more typical, no?

@@ -101,7 +101,6 @@ <h2 id="question">{{ t.pagecontent.question }}</h2>
</div>
</div>
</li>

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add this back please

Comment on lines 10 to 16
prefix: Homebrew won’t install files outside its prefix and you can place a Homebrew installation wherever you like.
createpackages: Trivially create your own Homebrew packages.
hack: It’s all Git and Ruby underneath, so hack away with the knowledge that you can easily revert your modifications and merge upstream updates.
prefix: Homebrew won’t install files outside its prefix, but you can place a Homebrew installation wherever you want.
createpackages: You can trivially create your own Homebrew packages.
hack: It’s just Git and Ruby underneath, you can hack away with the knowledge, easily revert your modifications, and merge upstream updates.
formula: "Homebrew formulae are simple Ruby scripts:"
editor: opens in $EDITOR!
editor: open it in $EDITOR!
complement: Homebrew complements macOS (or your Linux system). Install your RubyGems with <code>gem</code> and their dependencies with <code>brew</code>.
caskinstall: '“To install, drag this icon…” no more. <a href="https://formulae.brew.sh/cask/">Homebrew&nbsp;Cask</a> installs macOS apps, fonts and plugins and other non-open source software.'
caskinstall: '“To install, drag the icon…” not anymore, as <a href="https://formulae.brew.sh/cask/">Homebrew Casks</a> can install macOS apps, fonts and plugins and other non-open source software.'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please leave the English wording as-is, thanks, or open a dedicated PR to discuss

@65hz
Copy link
Contributor Author

65hz commented Dec 25, 2023

Sorry about the changes, did not know English was not meant to be changed.

I'll sync over the fork and make the changes to the Portuguese and Spanish pages.

@65hz 65hz closed this Dec 25, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants