Skip to content

Conversation

@thomasstorm
Copy link
Contributor

@thomasstorm thomasstorm commented Aug 15, 2025

Tiny PR that simply removes a few superfluous lines, and updates some default parameters. I fixes #127.

Checklist:

  • Issue has been created for change
  • Added docstrings and API docs for any new/modified user-facing classes and functions - dna
  • New/modified features documented in docs/source/* - dna
  • Changes documented in CHANGES.md
  • GitHub CI passes
  • Test coverage remains or increases (target 100%)

@thomasstorm thomasstorm self-assigned this Aug 15, 2025
@codecov
Copy link

codecov bot commented Aug 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.66%. Comparing base (1b4b033) to head (296593d).
⚠️ Report is 21 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #129      +/-   ##
==========================================
- Coverage   91.67%   91.66%   -0.01%     
==========================================
  Files          13       13              
  Lines        2812     2809       -3     
==========================================
- Hits         2578     2575       -3     
  Misses        234      234              

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

@thomasstorm thomasstorm merged commit 68a4583 into main Aug 20, 2025
5 checks passed
@thomasstorm thomasstorm deleted the thomas-127-fix_defaults branch August 20, 2025 09:25
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.

Don't use Auth0 in defaults anymore

2 participants