-
Notifications
You must be signed in to change notification settings - Fork 5k
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 doc about effects for capabilities #1842
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1842 +/- ##
===========================================
+ Coverage 35.90% 54.69% +18.78%
===========================================
Files 63 63
Lines 6637 6608 -29
Branches 1456 1581 +125
===========================================
+ Hits 2383 3614 +1231
+ Misses 4063 2696 -1367
- Partials 191 298 +107
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@LittleLittleCloud I changed the trigger back to pull_request to address a security requirement. |
OK sounds good, that might prevent PR from forked repo to consume openai secrets? |
Yes. So if a PR requires an openai test, it needs to be made from a branch within this repo. |
* add doc about effects for capabilities * remove unnecessary imports * improve doc * test * test location * polish * improve import of colored * termcolor * termcolor * use pull request for openai test
Why are these changes needed?
Related issue number
Checks