-
Notifications
You must be signed in to change notification settings - Fork 260
REQUEST: Repository maintenance on opentelemetry-rust #2746
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
Comments
oh, this looks really interesting there are potentially lots of automations that could benefit from this which we haven't been able to implement due to the security issues around cc @open-telemetry/docs-maintainers |
cc @open-telemetry/sig-project-infra-approvers |
Interestingly the docs folks are implementing something similar manually, see pros/cons here: https://github.com/open-telemetry/opentelemetry.io/pull/6894/files#r2091256900 |
I was (finally) able to get something like this working without relying on an external github app: It's definitely not as simple as using autofix.ci, but it doesn't require giving content write permission to a (relatively small) third party app. Let me know if you'd like to go this route and I can set up an opentelemetry org github app for your repo similar to https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/ec060ad52f9e0de9169108c27337fc2b4717b5f4/.github/workflows/auto-update-pull-request.yml#L40-L41 |
Affected Repository
https://github.com/open-telemetry/opentelemetry-rust
https://github.com/open-telemetry/opentelemetry-rust-contrib
Requested changes
Install autofix.ci GH App
https://autofix.ci/setup
Purpose
We would like to test out the autofix capabilities of the app to remove the additional work for contributors, when forgetting to format/lint their changes.
Expected Duration
4 weeks
Repository Maintainers
The text was updated successfully, but these errors were encountered: