You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@dolphinscheduler.apache.org by GitBox <gi...@apache.org> on 2022/11/29 10:52:40 UTC

[GitHub] [dolphinscheduler-sdk-python] zhongjiajie opened a new pull request, #35: [fix] Resource upload and add document

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

   <!--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)`.-->
   
   * Fix resource upload function related to apache/dolphinscheduler#12076
   * And some documentation about how to use resource uploading
   
   ref: apache/dolphinscheduler#13042
   
   ## Pull Request checklist
   
   I confirm that the following checklist has been completed.
   
   - [x] Add/Change **test cases** for the changes.
   - [x] Add/Change the related **documentation**, should also change `docs/source/config.rst` when you change file `default_config.yaml`.
   - [x] (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


[GitHub] [dolphinscheduler-sdk-python] zhongjiajie commented on pull request #35: [fix] Resource upload and add document

Posted by GitBox <gi...@apache.org>.
zhongjiajie commented on PR #35:
URL: https://github.com/apache/dolphinscheduler-sdk-python/pull/35#issuecomment-1330460337

   should be merged unless we related version 4.0.0


-- 
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


[GitHub] [dolphinscheduler-sdk-python] codecov-commenter commented on pull request #35: [fix] Resource upload and add document

Posted by GitBox <gi...@apache.org>.
codecov-commenter commented on PR #35:
URL: https://github.com/apache/dolphinscheduler-sdk-python/pull/35#issuecomment-1330446605

   # [Codecov](https://codecov.io/gh/apache/dolphinscheduler-sdk-python/pull/35?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report
   > Merging [#35](https://codecov.io/gh/apache/dolphinscheduler-sdk-python/pull/35?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (4fbc92d) into [main](https://codecov.io/gh/apache/dolphinscheduler-sdk-python/commit/2dfbb9b1582053a612ecc11f9fa373f11fe9cca6?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (2dfbb9b) will **increase** coverage by `0.00%`.
   > The diff coverage is `53.33%`.
   
   ```diff
   @@           Coverage Diff           @@
   ##             main      #35   +/-   ##
   =======================================
     Coverage   90.68%   90.68%           
   =======================================
     Files          59       59           
     Lines        2072     2073    +1     
   =======================================
   + Hits         1879     1880    +1     
     Misses        193      193           
   ```
   
   | Flag | Coverage Δ | |
   |---|---|---|
   | unittests | `90.68% <53.33%> (+<0.01%)` | :arrow_up: |
   
   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=The+Apache+Software+Foundation#carryforward-flags-in-the-pull-request-comment) to find out more.
   
   | [Impacted Files](https://codecov.io/gh/apache/dolphinscheduler-sdk-python/pull/35?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | Coverage Δ | |
   |---|---|---|
   | [src/pydolphinscheduler/core/workflow.py](https://codecov.io/gh/apache/dolphinscheduler-sdk-python/pull/35/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c3JjL3B5ZG9scGhpbnNjaGVkdWxlci9jb3JlL3dvcmtmbG93LnB5) | `81.86% <0.00%> (ø)` | |
   | [src/pydolphinscheduler/models/tenant.py](https://codecov.io/gh/apache/dolphinscheduler-sdk-python/pull/35/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c3JjL3B5ZG9scGhpbnNjaGVkdWxlci9tb2RlbHMvdGVuYW50LnB5) | `56.66% <0.00%> (ø)` | |
   | [src/pydolphinscheduler/core/resource.py](https://codecov.io/gh/apache/dolphinscheduler-sdk-python/pull/35/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c3JjL3B5ZG9scGhpbnNjaGVkdWxlci9jb3JlL3Jlc291cmNlLnB5) | `86.36% <33.33%> (ø)` | |
   | [src/pydolphinscheduler/constants.py](https://codecov.io/gh/apache/dolphinscheduler-sdk-python/pull/35/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c3JjL3B5ZG9scGhpbnNjaGVkdWxlci9jb25zdGFudHMucHk=) | `100.00% <100.00%> (ø)` | |
   | [src/pydolphinscheduler/core/task.py](https://codecov.io/gh/apache/dolphinscheduler-sdk-python/pull/35/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c3JjL3B5ZG9scGhpbnNjaGVkdWxlci9jb3JlL3Rhc2sucHk=) | `93.71% <100.00%> (+0.03%)` | :arrow_up: |
   
   :mega: We’re building smart automated test selection to slash your CI/CD build times. [Learn more](https://about.codecov.io/iterative-testing/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   


-- 
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


[GitHub] [dolphinscheduler-sdk-python] zhongjiajie merged pull request #35: [fix] Resource upload in dev branch

Posted by GitBox <gi...@apache.org>.
zhongjiajie merged PR #35:
URL: https://github.com/apache/dolphinscheduler-sdk-python/pull/35


-- 
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