-
Notifications
You must be signed in to change notification settings - Fork 32
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
Add import check #2146
Add import check #2146
Conversation
f092e0f
to
0d27462
Compare
Made a temporary commit to check if it works or not and the check works - https://github.com/valory-xyz/open-autonomy/actions/runs/7622558730/job/20760831758?pr=2146 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2146 +/- ##
=======================================
Coverage 94.52% 94.53%
=======================================
Files 259 259
Lines 15914 15914
=======================================
+ Hits 15043 15044 +1
+ Misses 871 870 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
The check failed. |
It was supposed to |
Ok, so you tested that it fails using f092e0f. |
Proposed changes
This PR add a clean installation check to avoid missing dependency issues from the framework
Fixes
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce? (A breaking change is a fix or feature that would cause existing functionality and APIs to not work as expected.)
Put an
x
in the box that appliesChecklist
Put an
x
in the boxes that apply.main
branch (left side). Also you should start your branch off ourmain
.