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

[es] feature: contributing - spanish translation #5058

Merged
merged 8 commits into from
Aug 27, 2024

Conversation

krol3
Copy link
Contributor

@krol3 krol3 commented Aug 18, 2024

Related the issue: #4887

@krol3 krol3 requested a review from a team August 18, 2024 16:24
@theletterf theletterf marked this pull request as draft August 19, 2024 12:51
@krol3
Copy link
Contributor Author

krol3 commented Aug 22, 2024

@ramrodo @theletterf @electrocucaracha, could you review please?

@krol3 krol3 marked this pull request as ready for review August 22, 2024 17:32
Signed-off-by: carolina valencia <krol3@users.noreply.github.com>
Copy link
Contributor

@ramrodo ramrodo left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@theletterf theletterf left a comment

Choose a reason for hiding this comment

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

Nice! Some suggestions.

I'm adding the default_lang_commit to all other PRs. To learn what that is about, see https://opentelemetry.io/docs/contributing/localization/#adding-default_lang_commit-to-new-pages

content/es/docs/contributing/_index.md Outdated Show resolved Hide resolved
content/es/docs/contributing/_index.md Outdated Show resolved Hide resolved
content/es/docs/contributing/_index.md Outdated Show resolved Hide resolved
@theletterf
Copy link
Member

@krol3 PTAL

content/es/docs/contributing/_index.md Show resolved Hide resolved
content/es/docs/contributing/_index.md Outdated Show resolved Hide resolved
content/es/docs/contributing/_index.md Outdated Show resolved Hide resolved
theletterf and others added 4 commits August 26, 2024 20:47
Co-authored-by: Victor Morales <chipahuac@hotmail.com>
Co-authored-by: Fabrizio Ferri-Benedetti <fferribenedetti@splunk.com>
Co-authored-by: Fabrizio Ferri-Benedetti <fferribenedetti@splunk.com>
Co-authored-by: Fabrizio Ferri-Benedetti <fferribenedetti@splunk.com>
@theletterf
Copy link
Member

/fix:all

@opentelemetrybot
Copy link
Collaborator

You triggered fix:all action run at https://github.com/open-telemetry/opentelemetry.io/actions/runs/10573598385

@theletterf theletterf added this pull request to the merge queue Aug 27, 2024
Merged via the queue into open-telemetry:main with commit aca2e07 Aug 27, 2024
17 checks passed
michael2893 pushed a commit to michael2893/opentelemetry.io that referenced this pull request Sep 8, 2024
Signed-off-by: carolina valencia <krol3@users.noreply.github.com>
Co-authored-by: carolina valencia <krol3@users.noreply.github.com>
Co-authored-by: Fabrizio Ferri-Benedetti <fferribenedetti@splunk.com>
Co-authored-by: Victor Morales <chipahuac@hotmail.com>
Co-authored-by: opentelemetrybot <107717825+opentelemetrybot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants