Skip to content

test: Activate resource test due to default local path#87

Merged
zhongjiajie merged 3 commits into
apache:mainfrom
zhongjiajie:ci-activate-test
Apr 14, 2023
Merged

test: Activate resource test due to default local path#87
zhongjiajie merged 3 commits into
apache:mainfrom
zhongjiajie:ci-activate-test

Conversation

@zhongjiajie

Copy link
Copy Markdown
Member

related PR: apache/dolphinscheduler@78e5569

Activate resource test due to default local path, related PR: apache/dolphinscheduler@78e5569

Brief Summary of The Change

Pull Request checklist

I confirm that the following checklist has been completed.

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

@zhongjiajie zhongjiajie added chore A collection of chore like CI, remove not used code dep-main-repo Dependent on repository apache/dolphinscheduler labels Apr 3, 2023
@zhongjiajie

Copy link
Copy Markdown
Member Author

should release after apache/dolphinscheduler:3.2.0

@codecov-commenter

codecov-commenter commented Apr 3, 2023

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.75%. Comparing base (2ef725e) to head (66d5872).
⚠️ Report is 40 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #87   +/-   ##
=======================================
  Coverage   90.75%   90.75%           
=======================================
  Files          63       63           
  Lines        2314     2314           
=======================================
  Hits         2100     2100           
  Misses        214      214           
Flag Coverage Δ
unittests 90.75% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

jieguangzhou
jieguangzhou previously approved these changes Apr 4, 2023

@jieguangzhou jieguangzhou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

[run-it]
[run-it]
@zhongjiajie zhongjiajie merged commit e166fc2 into apache:main Apr 14, 2023
@zhongjiajie zhongjiajie deleted the ci-activate-test branch April 14, 2023 06:23
@zhongjiajie zhongjiajie removed the dep-main-repo Dependent on repository apache/dolphinscheduler label Jan 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore A collection of chore like CI, remove not used code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants