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

Discover Affected Recordings #8391

Merged
merged 13 commits into from
Jun 10, 2024
Merged

Discover Affected Recordings #8391

merged 13 commits into from
Jun 10, 2024

Conversation

scbedd
Copy link
Member

@scbedd scbedd commented Jun 6, 2024

An easy script which automates:

  1. get latest assets under whatever path in the repos we want to (configured)
  2. grab those assets and dump them on disk
  3. rapidly search them for the contents we're looking for

This give us an easy way to be like

how many tests use this header?

Discovered todo:

  • fix the failing unit tests. Why are these failing with permission errors?
  • Revert passing of working directory to the scan and scanrepo functions. Totally unecessary given that WorkingDirectory is a PROPERTY OF THE CLASS.

@scbedd scbedd requested a review from a team as a code owner June 6, 2024 23:53
@scbedd scbedd self-assigned this Jun 7, 2024
…locate-involved-assets.ps1

Co-authored-by: Ben Broderick Phillips <ben@benbp.net>
@scbedd
Copy link
Member Author

scbedd commented Jun 10, 2024

/check-enforcer override

No CI checks for this build yet.

@scbedd scbedd merged commit 99c86b9 into Azure:main Jun 10, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants