-
Notifications
You must be signed in to change notification settings - Fork 44.6k
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
ci: fix classic ci #8338
ci: fix classic ci #8338
Conversation
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #8338 +/- ##
======================================
Coverage ? 49.62%
======================================
Files ? 144
Lines ? 8916
Branches ? 1240
======================================
Hits ? 4425
Misses ? 4344
Partials ? 147
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request. |
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly. |
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.
A million unrelated (formatting) changes, pls no
Don't worry i'll undo them all when I get the tests passing |
I didn't realise you had a PR for this already... but I merged one yesterday - does that fix what you are also trying to fix or completely different? |
@aarushik93 this PR fixes a couple of references to |
Imo your pr was great because it got the ones we use often working immediately |
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request. |
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly. |
This reverts commit c478d3c.
@aarushik93 ITS READY! |
Background
We moved from master -> dev for the mainline development branch without making sure it triggers the ci. This updates all the instances of development to dev for branch names. It does not attempt to verify the CI functions as needed under the new scheme
Changes 🏗️
Testing 🔍
Note
Only for the new autogpt platform, currently in autogpt_platform/