Skip to content

Comments

rf(script): Load config from file/env#3782

Merged
nellh merged 1 commit intoOpenNeuroOrg:masterfrom
effigies:script/config
Feb 20, 2026
Merged

rf(script): Load config from file/env#3782
nellh merged 1 commit intoOpenNeuroOrg:masterfrom
effigies:script/config

Conversation

@effigies
Copy link
Contributor

Updates to scripts/s3-delete-all-versions.py

  • Adds config loading from file/env. The old flags still work, but now you can explicitly opt in to reading them from the environment with --env or from a secrets file with --config.
  • Adds a dry-run flag
  • Uses Typer for the CLI since that's what I've been using lately and requires less looking up than sticking with argparse

Surprisingly large diff for what I was trying to do, mostly stuff copied from #3773, but running ruff increased the size.

@codecov
Copy link

codecov bot commented Feb 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 44.87%. Comparing base (942aa63) to head (4bab540).
⚠️ Report is 10 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3782      +/-   ##
==========================================
+ Coverage   42.60%   44.87%   +2.26%     
==========================================
  Files         642      642              
  Lines       34488    34497       +9     
  Branches     1557     1653      +96     
==========================================
+ Hits        14695    15480     +785     
+ Misses      19646    18880     -766     
+ Partials      147      137      -10     

☔ 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.

@nellh nellh merged commit 2a6234d into OpenNeuroOrg:master Feb 20, 2026
17 checks passed
@effigies effigies deleted the script/config branch February 20, 2026 18:19
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