Skip to content
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

fix: modify the error message returned #1709

Merged
merged 1 commit into from
Sep 7, 2021

Conversation

littlejiancc
Copy link
Contributor

What type of this PR

/kind bug

What this PR does / why we need it:

fix: modify the error message returned

Which issue(s) this PR fixes:

  • Fixes #your-issue_number
  • [Erda Cloud Issue Link](paste your link here)

Specified Reviewers:

/assign @sfwn

ChangeLog

Language Changelog
🇺🇸 English 修改返回的错误信息
🇨🇳 中文 modify the error message returned

Need cherry-pick to release versions?

Add comment like /cherry-pick release/1.0 when this PR is merged.

For details on the cherry pick process, see the cherry pick requests section under CONTRIBUTING.md.

@erda-bot erda-bot requested a review from sfwn September 7, 2021 08:43
@codecov
Copy link

codecov bot commented Sep 7, 2021

Codecov Report

Merging #1709 (e32cf6a) into master (7d733b0) will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1709   +/-   ##
=======================================
  Coverage   13.67%   13.67%           
=======================================
  Files        1108     1108           
  Lines      114857   114865    +8     
=======================================
+ Hits        15703    15708    +5     
+ Misses      97603    97599    -4     
- Partials     1551     1558    +7     
Impacted Files Coverage Δ
modules/pipeline/services/pipelinesvc/snippet.go 42.52% <0.00%> (+0.96%) ⬆️

@sfwn sfwn merged commit 2b01c6b into erda-project:master Sep 7, 2021
@littlejiancc littlejiancc deleted the feature/gittar-error branch September 24, 2021 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants