add temporary script to replace utils/Rt_review_exclusions.R#377
Merged
natemcintosh merged 11 commits intomainfrom Jan 2, 2026
Merged
add temporary script to replace utils/Rt_review_exclusions.R#377natemcintosh merged 11 commits intomainfrom
natemcintosh merged 11 commits intomainfrom
Conversation
for more information, see https://pre-commit.ci
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
PatrickTCorbett
temporarily deployed
to
production
GitHub Actions
Inactive
PatrickTCorbett
temporarily deployed
to
production
GitHub Actions
Inactive
Contributor
|
Thank you for your contribution @natemcintosh 🚀! Your pkgdown-site is ready for download 👉 here 👈! |
natemcintosh
temporarily deployed
to
production
GitHub Actions
Inactive
natemcintosh
temporarily deployed
to
production
GitHub Actions
Inactive
natemcintosh
temporarily deployed
to
production
GitHub Actions
Inactive
natemcintosh
temporarily deployed
to
production
GitHub Actions
Inactive
natemcintosh
temporarily deployed
to
production
GitHub Actions
Inactive
natemcintosh
temporarily deployed
to
production
GitHub Actions
Inactive
natemcintosh
approved these changes
Dec 30, 2025
Collaborator
|
Reviewed by @PatrickTCorbett but I'm approving due to 2fa issues. |
PatrickTCorbett
commented
Jan 2, 2026
Collaborator
Author
PatrickTCorbett
left a comment
PatrickTCorbett
left a comment
There was a problem hiding this comment.
This looks good and tracks with what the R code was doing (got 2FA issues resolved)
natemcintosh
deleted the
376-handle-change-in-secondary-app-authentication
branch
natemcintosh
temporarily deployed
to
production
GitHub Actions
Inactive
natemcintosh
temporarily deployed
to
production
GitHub Actions
Inactive
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
NEW:
Add a uv-managed python script to read a local Rt review file, process it into point and state exclusions, and upload them to blob.
Run
uv run utils/exclusions_file_parsing.py -hto see what arguments to use. All dependencies and the python version used will be managed byuv.If you get a message like "No interpreter found for Python ....", try running
and then the script again.
I removed the R version of this script to reduce confusion.
OLD:
Need to edit the script that pulls from SharePoint. Instead SOP will detail user to download Excel file to local download folder and script will parse it locally before uploading to blob