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

Internationalization and localization improvements #6400

Merged
merged 27 commits into from
May 26, 2023
Merged

Internationalization and localization improvements #6400

merged 27 commits into from
May 26, 2023

Conversation

JE-Chen
Copy link
Contributor

@JE-Chen JE-Chen commented May 24, 2023

#2642
#6363

Fix origin problem like link, grammar.
image
image

JE-Chen and others added 3 commits May 24, 2023 09:13
Add back .[adoc, images]
Edit _chapter.yml
Refactor and Fix PR #2642 Problem
@JE-Chen JE-Chen requested a review from a team as a code owner May 24, 2023 02:12
@probot-autolabeler probot-autolabeler bot added the documentation Jenkins documentation, including user and developer docs, solution pages, etc. label May 24, 2023
* Replace old name to new name Jenkins Development Support
* Add image
* Refactor doc
Update doc, old plugin option can't use now
Copy link
Contributor

@MarkEWaite MarkEWaite left a comment

Choose a reason for hiding this comment

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

Thanks so much for the pull request! I've made several proposed changes for your consideration. I haven't completed my review yet, but wanted to provide these initial comments so that you can continue the pull request and we can further improve it.

Refactor the doc:
Remove too old data.
Hide  NetBeans plugin intro.
etc.
Copy link
Contributor

@gounthar gounthar left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution.
Please feel free to make changes according to your preferences, including applying the one sentence per line rule.

Copy link
Contributor

@gounthar gounthar left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution.
Please feel free to make changes according to your preferences, including applying the one sentence per line rule.

@MarkEWaite MarkEWaite changed the title Refactor and fix PR #2642 problem Internationalization and localization improvements May 24, 2023
Copy link
Contributor

@MarkEWaite MarkEWaite left a comment

Choose a reason for hiding this comment

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

@JE-Chen thanks so much for the contributions.

I think that some of this content is covered in pages that already exist. I also think that this content would be better if we placed it on the top level page at http://www.jenkins.io/doc/developer/internationalization/ and marked that page as no longer being a "Work in progress".

Would it be OK with you if I submitted additional commits to this pull request in order to show what I mean by those ideas? If you find that you don't like the ideas, we can revert or delete my commits to the pull request

Copy link
Contributor

@MarkEWaite MarkEWaite left a comment

Choose a reason for hiding this comment

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

@JE-Chen thanks so much for the contributions.

I think that some of this content is covered in pages that already exist. I also think that this content would be better if we placed it on the top level page at http://www.jenkins.io/doc/developer/internationalization/ and marked that page as no longer being a "Work in progress".

Would it be OK with you if I submitted additional commits to this pull request in order to show what I mean by those ideas? If you find that you don't like the ideas, we can revert or delete my commits to the pull request

They are closely related and belong together
Mark the internationalization and localization page as no longer being a
work in progress.  These changes make it complete enough.

Still needs more work to bring additional content from @JE-Chen
@MarkEWaite
Copy link
Contributor

MarkEWaite commented May 24, 2023

@JE-Chen I submitted my ideas as additional commits in a draft pull request

If the general structure of those changes is acceptable to you, I'll bring more of your content from the proposed files into the new structure and submit them all on this branch.

You can view the preview of that proposal as:

@JE-Chen
Copy link
Contributor Author

JE-Chen commented May 25, 2023

@JE-Chen I submitted my ideas as additional commits in a draft pull request

If the general structure of those changes is acceptable to you, I'll bring more of your content from the proposed files into the new structure and submit them all on this branch.

You can view the preview of that proposal as:

OK, Thanks.

Refactor
one sentence per line rule.
titles.
link.
Replace :
In this case, first you must write `+foo.properties+` for the default
locale:
with:
In this case, first, you must write `+foo.properties+` for the default locale:
Replace:
You can pass multiple parameters, separating them with ','
(just like a function call), and from property files you can reference
them as \{0}, \{1}, etc., by following
https://docs.oracle.com/en/java/javase/17/docs/api/java.base/java/text/MessageFormat.html[the
with:
You can pass multiple parameters, separating them with ',' (just like a function call), and from property files, you can reference them as \{0}, \{1}, etc., by following link:https://docs.oracle.com/en/java/javase/17/docs/api/java.base/java/text/MessageFormat.html[the
MessageFormat class convention].
Refactor style.
Fix an format, I forgot change.
Copy link
Contributor

@MarkEWaite MarkEWaite left a comment

Choose a reason for hiding this comment

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

I've completed my changes. @JE-Chen does this look good enough to you?

@JE-Chen
Copy link
Contributor Author

JE-Chen commented May 26, 2023

@MarkEWaite It's good to me. Thanks.

@MarkEWaite MarkEWaite merged commit 00546ec into jenkins-infra:master May 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Jenkins documentation, including user and developer docs, solution pages, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants