Skip to content

Conversation

@MrAlias
Copy link
Contributor

@MrAlias MrAlias commented Jan 28, 2025

  • Use the local user to install and run so that root does not own the artifacts.
  • Move the python image into a place renovate can maintain it.

@MrAlias MrAlias added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Jan 28, 2025
Use the local user to install and run so that root does not own the
artifacts.

Move the python image into a place renovate can maintain it.
@MrAlias MrAlias force-pushed the codespell-ownership branch from d29d8ec to fbffc01 Compare January 28, 2025 22:16
@MrAlias MrAlias marked this pull request as ready for review January 28, 2025 22:22
@codecov
Copy link

codecov bot commented Jan 28, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.2%. Comparing base (d10a36b) to head (44d62e7).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##            main   #6223     +/-   ##
=======================================
- Coverage   82.2%   82.2%   -0.1%     
=======================================
  Files        273     273             
  Lines      23744   23744             
=======================================
- Hits       19539   19537      -2     
- Misses      3856    3858      +2     
  Partials     349     349             

see 1 file with indirect coverage changes

@MrAlias MrAlias merged commit 1994f63 into open-telemetry:main Jan 29, 2025
31 of 32 checks passed
@MrAlias MrAlias deleted the codespell-ownership branch January 29, 2025 22:20
MrAlias added a commit to MrAlias/opentelemetry-go-contrib that referenced this pull request Jan 29, 2025
Copy of open-telemetry/opentelemetry-go#6223

- Use the local user to install and run so that root does not own the
  artifacts.
- Move the python image into a place renovate can maintain it.
@MrAlias MrAlias added this to the v1.35.0 milestone Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants