Skip to content

Conversation

kakj-go
Copy link
Contributor

@kakj-go kakj-go commented Jun 30, 2021

What type of this PR

Add one of the following kinds:
/kind feature

What this PR does / why we need it:

dop and core-services same function move to apistructs package

@kakj-go kakj-go changed the title appModeCheck move api structs appModeCheck move to apistructs package Jun 30, 2021
@kakj-go kakj-go force-pushed the feature/move-app-mod-check branch from 4ae601a to 0f17463 Compare June 30, 2021 07:21
@codecov
Copy link

codecov bot commented Jun 30, 2021

Codecov Report

Merging #732 (cf9ce18) into master (d50f7bc) will decrease coverage by 0.00%.
The diff coverage is 83.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #732      +/-   ##
==========================================
- Coverage   10.11%   10.11%   -0.01%     
==========================================
  Files         935      931       -4     
  Lines       87700    87689      -11     
==========================================
- Hits         8875     8873       -2     
+ Misses      77898    77891       -7     
+ Partials      927      925       -2     
Impacted Files Coverage Δ
modules/dop/endpoints/application.go 0.00% <0.00%> (ø)
apistructs/application.go 45.45% <100.00%> (+45.45%) ⬆️
modules/dop/utils/storageutil.go
pkg/mock/mock.go 28.76% <0.00%> (+4.10%) ⬆️

@sfwn
Copy link
Member

sfwn commented Jun 30, 2021

/approve

@erda-bot erda-bot merged commit f6ce890 into erda-project:master Jun 30, 2021
@kakj-go kakj-go deleted the feature/move-app-mod-check branch July 22, 2021 02:45
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.

3 participants