Skip to content

fix: Fix the problem that the switch task cannot parse the date param…#7884

Closed
JohnZp wants to merge 4 commits into
apache:devfrom
JohnZp:dev
Closed

fix: Fix the problem that the switch task cannot parse the date param…#7884
JohnZp wants to merge 4 commits into
apache:devfrom
JohnZp:dev

Conversation

@JohnZp
Copy link
Copy Markdown
Contributor

@JohnZp JohnZp commented Jan 7, 2022

Purpose of the pull request

fix and improvement

Brief change log

  • Fix the problem that the switch task cannot parse the date params
  • Fix that the resource path cannot be returned correctly when clicking on it
  • Fix the task list of the selected project that cannot be correctly requested when adding a dependency to the dependency
  • Added resource re-upload button

Verify this pull request

This pull request is code cleanup without any test coverage.

@zhuangchong zhuangchong added the first time contributor First-time contributor label Jan 8, 2022
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jan 8, 2022

Codecov Report

❌ Patch coverage is 0% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 40.99%. Comparing base (361d68d) to head (5bbf05f).
⚠️ Report is 3184 commits behind head on dev.

Files with missing lines Patch % Lines
...server/master/runner/task/SwitchTaskProcessor.java 0.00% 9 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##                dev    #7884      +/-   ##
============================================
- Coverage     41.07%   40.99%   -0.08%     
+ Complexity     3702     3693       -9     
============================================
  Files           637      637              
  Lines         27058    27047      -11     
  Branches       3073     3071       -2     
============================================
- Hits          11113    11088      -25     
- Misses        14873    14889      +16     
+ Partials       1072     1070       -2     

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@sonarqubecloud
Copy link
Copy Markdown

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 3 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@songjianet
Copy link
Copy Markdown
Member

Thank you for your contribution, but since this issue has not been dealt with for a long time, I will close it. If you still have problems in the future, you can re-submit a new issue

@songjianet songjianet closed this Mar 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

first time contributor First-time contributor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants