-
Notifications
You must be signed in to change notification settings - Fork 75
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
Run auto-tick
with a single package only
#2813
base: main
Are you sure you want to change the base?
Conversation
2ed9925
to
78ca5a9
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2813 +/- ##
==========================================
- Coverage 77.80% 77.71% -0.09%
==========================================
Files 128 128
Lines 13841 13880 +39
==========================================
+ Hits 10769 10787 +18
- Misses 3072 3093 +21 ☔ View full report in Codecov by Sentry. |
a694936
to
e645003
Compare
e645003
to
8cd8a0d
Compare
Since it's still very burdensome to add a fully-fledged integration/end-to-end test for the auto-tick command, I decided to skip it for now. When we deal with #261, we might also have a look at testing the dry-run mode. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I made some initial comments. package
needs to be changed to feedstock
everywhere. After the existing comments are addressed, I'll take another look.
45050de
to
3d2ed2a
Compare
cb5edda
to
fcd48e6
Compare
2c9259f
to
05bdf9a
Compare
pre-commit.ci autofix |
Co-authored-by: Pavel Zwerschke <[email protected]>
Co-authored-by: Pavel Zwerschke <[email protected]>
Co-authored-by: Pavel Zwerschke <[email protected]>
for more information, see https://pre-commit.ci
5255be0
to
3acde80
Compare
This PR adds an argument to the
auto-tick
command allowing it to be run with a single package only for easier debugging.Cc @pavelzw
Add an Integration Test(see comment)