Skip to content

Conversation

arrdem
Copy link
Collaborator

@arrdem arrdem commented Jul 23, 2025

This patch integrates tools_telemetry into rules_py. Depending on rules_py will register a dependency into the telemetry system and produce a usage report when external repositories are invalidated.

Changes are visible to end-users: yes

  • Suggested release notes appear below: yes

aspect_tools_telemetry is now used for coarse grained usage tracking.

Test plan

N/A.

@arrdem arrdem force-pushed the arrdem/feat-telemetry branch from b48cc05 to 22afcb6 Compare July 23, 2025 17:22
Copy link

aspect-workflows bot commented Jul 23, 2025

Test

All tests were cache hits

41 tests (100.0%) were fully cached saving 1m 9s.

@arrdem arrdem requested a review from alexeagle July 23, 2025 17:22
Copy link

github-actions bot commented Jul 23, 2025

e2e/use_release folder: LCOV of commit 12617ff during CI #1896

Summary coverage rate:
  lines......: 100.0% (2 of 2 lines)
  functions..: 100.0% (1 of 1 function)
  branches...: no data found

Files changed coverage rate: n/a

- Add tools_telemetry as a dependency of rules_py
- Make the rules_py toolchain registration perform a phone home
@arrdem arrdem force-pushed the arrdem/feat-telemetry branch from e500a59 to 12617ff Compare July 23, 2025 19:31
Copy link
Member

@alexeagle alexeagle left a comment

Choose a reason for hiding this comment

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

🚢

@arrdem arrdem merged commit e5168aa into main Jul 24, 2025
16 checks passed
@arrdem arrdem deleted the arrdem/feat-telemetry branch July 24, 2025 03:47
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.

2 participants