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

Enable Prettier check over instrumentation/java #2363

Merged

Conversation

chalin
Copy link
Contributor

@chalin chalin commented Feb 19, 2023

Preview: https://deploy-preview-2363--opentelemetry.netlify.app/docs/instrumentation/java/

@chalin chalin requested review from a team February 19, 2023 16:14
@chalin
Copy link
Contributor Author

chalin commented Feb 20, 2023

@cartermp @open-telemetry/javascript-maintainers - is this good to go?

@pichlermarc
Copy link
Member

@chalin did you mean to ping @open-telemetry/java-maintainers? 🙂

@chalin
Copy link
Contributor Author

chalin commented Feb 20, 2023

@pichlermarc - oops, yes, sorry!
Pinging @open-telemetry/java-maintainers!

@chalin chalin force-pushed the chalin-im-prettier-over-java-2023-02-19 branch from ef80fa4 to 5f66092 Compare February 20, 2023 13:12
@cartermp
Copy link
Contributor

Yep, this should be good to go now.

@cartermp cartermp merged commit 1ff41b1 into open-telemetry:main Feb 20, 2023
@cartermp
Copy link
Contributor

Also, maybe the JS team should be js-maintiners/js-approvers since this has happened a few times

@chalin chalin deleted the chalin-im-prettier-over-java-2023-02-19 branch February 20, 2023 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants