-
Notifications
You must be signed in to change notification settings - Fork 23
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
reorganized weeklies into dedicated workflow like dailies #1088
Conversation
@CodyCBakerPhD, for my own understanding, what is the reason that |
The base image for rclone is not assumed to update that regularly or that its updates are not that important Could maybe even go down to monthly Main edge case benefit is that you gain access to new service providers (but most of our expected usage is drive/dropbox anyway) |
Ok, sounds good. |
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.
Should we put the formatwise gallyer tests here as well?
Also, maybe the dev tests? They are on the dailies but we won't be able to fix them on a daily basis so once you get the notification once you will keep getting the notification every day. I guess that's your call.
Those are disabled, so I didn't include them. |
No, I definitely want to know about the dev tests on a daily basis. |
OK, I think we should activate them. But that's outside of the scope of this PR. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1088 +/- ##
=======================================
Coverage 90.44% 90.44%
=======================================
Files 129 129
Lines 8055 8050 -5
=======================================
- Hits 7285 7281 -4
+ Misses 770 769 -1
Flags with carried forward coverage won't be shown. Click here to find out more. |
Continuation of #1081 for weekly workflows.