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.

Sign up for GitHub

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

Jump to bottom

[pt] Add content/pt/docs/concepts/instrumentation/code-based.md #5093

Merged

theletterf merged 31 commits into open-telemetry:main from janssenlima:pt-br_code-based

Aug 26, 2024

Contributor

janssenlima commented Aug 22, 2024

Add localization instrumentation code-based.

Issue: #4922


          [pt] Add content/pt/docs/concepts/instrumentation/code-based.md

ccf779f

janssenlima requested a review from a team

August 22, 2024 23:19

opentelemetrybot requested review from a team

August 22, 2024 23:19

github-actions bot added the lang:pt label

emdneto reviewed

View reviewed changes

content/pt/docs/concepts/instrumentation/code-based.md Outdated Show resolved Hide resolved

emdneto reviewed

View reviewed changes

content/pt/docs/concepts/instrumentation/code-based.md Outdated Show resolved Hide resolved

emdneto reviewed

View reviewed changes

content/pt/docs/concepts/instrumentation/code-based.md Outdated Show resolved Hide resolved

EzzioMoreira reviewed

View reviewed changes

content/pt/docs/concepts/instrumentation/code-based.md Outdated Show resolved Hide resolved

EzzioMoreira reviewed

View reviewed changes

content/pt/docs/concepts/instrumentation/code-based.md Outdated Show resolved Hide resolved


          Update content/pt/docs/concepts/instrumentation/code-based.md

Co-authored-by: Emídio Neto <9735060+emdneto@users.noreply.github.com>

opentelemetrybot requested a review from a team

August 23, 2024 11:28

janssenlima and others added 7 commits

August 23, 2024 08:28


          Update content/pt/docs/concepts/instrumentation/code-based.md

10c54e0

Co-authored-by: Emídio Neto <9735060+emdneto@users.noreply.github.com>


          Update content/pt/docs/concepts/instrumentation/code-based.md

bfac31b

Co-authored-by: Ezzio Moreira <enzziom@gmail.com>


          Update content/pt/docs/concepts/instrumentation/code-based.md

0a3c527

Co-authored-by: Ezzio Moreira <enzziom@gmail.com>


          Merge branch 'main' into pt-br_code-based

7783a02


          Update code-based.md

4d78f16


          Update code-based.md

b26c9ed


          Fix file format

89d645c

emdneto approved these changes

View reviewed changes

Member

emdneto left a comment

Thanks

Member

emdneto commented Aug 23, 2024

/fix:all

Collaborator

opentelemetrybot commented Aug 23, 2024

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

opentelemetrybot and others added 2 commits

August 23, 2024 13:52


          Results from /fix:all

3abd8a1


          Tweak front matter

f7048e5

opentelemetrybot requested a review from a team

August 23, 2024 15:17

lgfa29 reviewed

View reviewed changes

Contributor

lgfa29 left a comment

Ótimo trabalho @janssenlima!

Deixei algumas ideias e sugestões 🙂

content/pt/docs/concepts/instrumentation/code-based.md Outdated Show resolved Hide resolved

content/pt/docs/concepts/instrumentation/code-based.md Outdated Show resolved Hide resolved

content/pt/docs/concepts/instrumentation/code-based.md Outdated Show resolved Hide resolved

content/pt/docs/concepts/instrumentation/code-based.md Outdated Show resolved Hide resolved

content/pt/docs/concepts/instrumentation/code-based.md Outdated Show resolved Hide resolved

content/pt/docs/concepts/instrumentation/code-based.md Outdated Show resolved Hide resolved

content/pt/docs/concepts/instrumentation/code-based.md Outdated Show resolved Hide resolved

content/pt/docs/concepts/instrumentation/code-based.md Outdated Show resolved Hide resolved

content/pt/docs/concepts/instrumentation/code-based.md Outdated Show resolved Hide resolved

content/pt/docs/concepts/instrumentation/code-based.md Outdated Show resolved Hide resolved

Contributor

chalin commented Aug 23, 2024

@emdneto et al.: is this ready to be merged (and have any remaining comments addressed in a followup PR)?

edsoncelio mentioned this pull request

[pt] Portuguese page localization #4922

Open

26 tasks

Member

emdneto commented Aug 23, 2024 •

edited

Loading

@chalin I will move this PR to draft while @janssenlima addresses the suggestions. There are important fixes that we missed at the first look

emdneto marked this pull request as draft

August 23, 2024 17:37

janssenlima and others added 4 commits

August 23, 2024 16:53


          Update content/pt/docs/concepts/instrumentation/code-based.md

e3bfe3c

Co-authored-by: Luiz Aoqui <lgfa29@gmail.com>


          Update content/pt/docs/concepts/instrumentation/code-based.md

de28822

Co-authored-by: Luiz Aoqui <lgfa29@gmail.com>


          Update content/pt/docs/concepts/instrumentation/code-based.md

48a132e

Co-authored-by: Luiz Aoqui <lgfa29@gmail.com>


          Update content/pt/docs/concepts/instrumentation/code-based.md

ce1ab8e

Co-authored-by: Luiz Aoqui <lgfa29@gmail.com>

janssenlima marked this pull request as ready for review

August 23, 2024 19:59


          Merge branch 'main' into pt-br_code-based

f7a6af5

janssenlima marked this pull request as draft

August 23, 2024 23:44

janssenlima and others added 6 commits

August 24, 2024 10:36


          Update content/pt/docs/concepts/instrumentation/code-based.md

a5f1c41

Co-authored-by: Luiz Aoqui <lgfa29@gmail.com>


          Update content/pt/docs/concepts/instrumentation/code-based.md

8f9fcf9

Co-authored-by: Luiz Aoqui <lgfa29@gmail.com>


          Update content/pt/docs/concepts/instrumentation/code-based.md

3ef7347

Co-authored-by: Luiz Aoqui <lgfa29@gmail.com>


          Update content/pt/docs/concepts/instrumentation/code-based.md

cec51e8

Co-authored-by: Luiz Aoqui <lgfa29@gmail.com>


          Update content/pt/docs/concepts/instrumentation/code-based.md

7eb7fe3

Co-authored-by: Luiz Aoqui <lgfa29@gmail.com>


          Update content/pt/docs/concepts/instrumentation/code-based.md

0e07e34

Co-authored-by: Luiz Aoqui <lgfa29@gmail.com>

Contributor Author

janssenlima commented Aug 24, 2024

@chalin I will move this PR to draft while @janssenlima addresses the suggestions. There are important fixes that we missed at the first look

@emdneto, fiz os ajustes. Só tem uma thread ali em discussão para fechar a documentação.


          Update code-based.md

ab04256

Title changed according to alignment.

janssenlima marked this pull request as ready for review

August 26, 2024 02:01


          Merge branch 'main' into pt-br_code-based

05db1fd

Contributor Author

janssenlima commented Aug 26, 2024

@emdneto, fiz todas as correções.

Member

emdneto commented Aug 26, 2024

/fix:all

Collaborator

opentelemetrybot commented Aug 26, 2024

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

Collaborator

opentelemetrybot commented Aug 26, 2024

fix:all run failed, please check https://github.com/open-telemetry/opentelemetry.io/actions/runs/10558843861 for details


          Fix code style issues with Prettier

a8d84ce

emdneto approved these changes

View reviewed changes

theletterf approved these changes

View reviewed changes

theletterf added this pull request to the merge queue

Merged via the queue into open-telemetry:main with commit 5459f7e

17 checks passed

janssenlima deleted the pt-br_code-based branch

August 31, 2024 19:07

edsoncelio mentioned this pull request

[pt] Add pt/content/docs/concepts/instrumentation #5098

Closed

michael2893 pushed a commit to michael2893/opentelemetry.io that referenced this pull request


          [pt] Add content/pt/docs/concepts/instrumentation/code-based.md (open…

f983cdf

…-telemetry#5093)

Co-authored-by: Emídio Neto <9735060+emdneto@users.noreply.github.com>
Co-authored-by: Ezzio Moreira <enzziom@gmail.com>
Co-authored-by: opentelemetrybot <107717825+opentelemetrybot@users.noreply.github.com>
Co-authored-by: Patrice Chalin <chalin@users.noreply.github.com>
Co-authored-by: Luiz Aoqui <lgfa29@gmail.com>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

lgfa29 lgfa29 left review comments

edsoncelio edsoncelio left review comments

marcelobaptista marcelobaptista left review comments

EzzioMoreira EzzioMoreira left review comments

theletterf theletterf approved these changes

emdneto emdneto approved these changes

Labels