-
Notifications
You must be signed in to change notification settings - Fork 175
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
doc: Update the contributing guidance #4222
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4222 +/- ##
=======================================
Coverage 76.69% 76.69%
=======================================
Files 759 759
Lines 41788 41788
Branches 9603 9603
=======================================
+ Hits 32049 32051 +2
+ Misses 8639 8638 -1
+ Partials 1100 1099 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
Looks good to me
…ghts#4232) - when collecting in py26 env, if the foreach_execute specs depends on DatasourceProvider, the provider will be in unicode type and causes the followed validate() fail - this change converted the unicode to string by force for all 'cmd' passed to CommandOutputProvider Signed-off-by: Xiangce Liu <xiangceliu@redhat.com> rh-pre-commit.version: 2.3.1 rh-pre-commit.check-secrets: ENABLED
35f3eb9
to
2e1d000
Compare
Signed-off-by: Xiangce Liu <xiangceliu@redhat.com> rh-pre-commit.version: 2.3.1 rh-pre-commit.check-secrets: ENABLED
2e1d000
to
9f7640f
Compare
378129c
to
01e540a
Compare
Signed-off-by: Xiangce Liu <xiangceliu@redhat.com> rh-pre-commit.version: 2.3.1 rh-pre-commit.check-secrets: ENABLED
01e540a
to
55375dd
Compare
Signed-off-by: Xiangce Liu xiangceliu@redhat.com
rh-pre-commit.version: 2.3.1
rh-pre-commit.check-secrets: ENABLED
All Pull Requests:
Check all that apply:
Complete Description of Additions/Changes:
Add your description here