Skip to content

Conversation

@singhd789
Copy link
Collaborator

@singhd789 singhd789 commented Nov 20, 2025

Describe your changes

  • Adds --force-checkout option to fre make: if the checkout script already exists but the user made a change to the yaml, this option can be used to remove and re-create the checkout script
  • removes verbose options at fre make checkout-script level (already implemented at fre level)
  • fixes up some format across files
  • updates logging level format

Issue ticket number and link (if applicable)

#674

Checklist before requesting a review

  • I ran my code
  • I tried to make my code readable
  • I tried to comment my code
  • I wrote a new test, if applicable
  • I wrote new instructions/documentation, if applicable
  • I ran pytest and inspected it's output
  • I ran pylint and attempted to implement some of it's feedback
  • No print statements; all user-facing info uses logging module

@codecov
Copy link

codecov bot commented Dec 9, 2025

Codecov Report

❌ Patch coverage is 88.23529% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.28%. Comparing base (f3155e8) to head (69b5ffb).

Files with missing lines Patch % Lines
fre/make/create_checkout_script.py 85.71% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #679      +/-   ##
==========================================
- Coverage   85.32%   85.28%   -0.04%     
==========================================
  Files          68       68              
  Lines        4490     4507      +17     
==========================================
+ Hits         3831     3844      +13     
- Misses        659      663       +4     
Flag Coverage Δ
unittests 85.28% <88.23%> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@singhd789 singhd789 linked an issue Dec 9, 2025 that may be closed by this pull request
@singhd789 singhd789 marked this pull request as ready for review December 30, 2025 18:26
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.

Add force-checkout option in fre make subtool

2 participants