You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@dolphinscheduler.apache.org by "zhongjiajie (via GitHub)" <gi...@apache.org> on 2023/12/25 10:54:47 UTC

[PR] fix: Integration test error [dolphinscheduler-sdk-python]

zhongjiajie opened a new pull request, #129:
URL: https://github.com/apache/dolphinscheduler-sdk-python/pull/129

   related change: https://github.com/apache/dolphinscheduler/pull/15254
   
   [run-it]
   
   <!--Thanks for you contribute to Apache DolphinScheduler Python API, You can see more detail about contributing in https://github.com/apache/dolphinscheduler-sdk-python/DEVELOP.md .-->
   
   ## Brief Summary of The Change
   
   <!--Please include `fixes: #XXXX(ISSUE_NUMBER)` to automatically close any corresponding issue when the pull request is merged. Alternatively if not fully closed you can say `related: #XXXX(ISSUE_NUMBER)`.-->
   
   ## Pull Request checklist
   
   I confirm that the following checklist has been completed.
   
   - [x] Add/Change **test cases** for the changes.
   - [ ] Add/Change the related **documentation**, should also change `docs/source/config.rst` when you change file `default_config.yaml`.
   - [ ] (Optional) Add your change to `UPDATING.md` when it is an incompatible change.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@dolphinscheduler.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] fix: Integration test error [dolphinscheduler-sdk-python]

Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #129:
URL: https://github.com/apache/dolphinscheduler-sdk-python/pull/129#issuecomment-1868927218

   ## [Codecov](https://app.codecov.io/gh/apache/dolphinscheduler-sdk-python/pull/129?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
   All modified and coverable lines are covered by tests :white_check_mark:
   > Comparison is base [(`e1d1527`)](https://app.codecov.io/gh/apache/dolphinscheduler-sdk-python/commit/e1d15274a6ffc2cf2ee3236d3ba3e8a0b1cf70d3?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) 90.86% compared to head [(`bd49758`)](https://app.codecov.io/gh/apache/dolphinscheduler-sdk-python/pull/129?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) 90.86%.
   
   
   <details><summary>Additional details and impacted files</summary>
   
   
   ```diff
   @@           Coverage Diff           @@
   ##             main     #129   +/-   ##
   =======================================
     Coverage   90.86%   90.86%           
   =======================================
     Files          64       64           
     Lines        2354     2354           
   =======================================
     Hits         2139     2139           
     Misses        215      215           
   ```
   
   | [Flag](https://app.codecov.io/gh/apache/dolphinscheduler-sdk-python/pull/129/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
   |---|---|---|
   | [unittests](https://app.codecov.io/gh/apache/dolphinscheduler-sdk-python/pull/129/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `90.86% <ø> (ø)` | |
   
   Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment) to find out more.
   
   
   </details>
   
   [:umbrella: View full report in Codecov by Sentry](https://app.codecov.io/gh/apache/dolphinscheduler-sdk-python/pull/129?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).   
   :loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@dolphinscheduler.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] fix: Integration test error [dolphinscheduler-sdk-python]

Posted by "zhongjiajie (via GitHub)" <gi...@apache.org>.
zhongjiajie merged PR #129:
URL: https://github.com/apache/dolphinscheduler-sdk-python/pull/129


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@dolphinscheduler.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org