Skip to content

Conversation

@amrit110
Copy link
Member

This pull request updates the scripts/collect_coder_analytics.py script to improve how workspaces are fetched and filtered, specifically by integrating Firestore to retrieve team mappings and using those mappings to exclude workspaces belonging to certain teams from analytics. The changes also update dependency requirements and error messaging.

Integration with Firestore and workspace filtering:

  • Added a new get_team_mappings function to fetch participant team mappings from Firestore, mapping GitHub handles to team names.
  • Updated the fetch_workspaces function to accept team mappings and filter out workspaces owned by users in excluded teams (facilitators and Unassigned).
  • Modified the main workflow to first fetch team mappings from Firestore and then use them to filter workspaces for analytics.

Dependency and error handling improvements:

  • Updated the import block to require both google-cloud-storage and google-cloud-firestore, and improved the error message to reflect both dependencies.

@amrit110 amrit110 self-assigned this Dec 10, 2025
@amrit110 amrit110 added the bug Something isn't working label Dec 10, 2025
@codecov
Copy link

codecov bot commented Dec 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.09%. Comparing base (b6e4f05) to head (438c03e).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #42   +/-   ##
=======================================
  Coverage   88.09%   88.09%           
=======================================
  Files           3        3           
  Lines         462      462           
=======================================
  Hits          407      407           
  Misses         55       55           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@amrit110 amrit110 merged commit a0ddf68 into main Dec 10, 2025
6 checks passed
@amrit110 amrit110 deleted the filter_out_non_teams branch December 10, 2025 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants