fabric8-services / fabric8-wit

wit stands for Work Item Tracker
http://devdoc.almighty.io/
Apache License 2.0
45 stars 86 forks source link

WIP: Rename jsonb field keys #2293

Closed DhritiShikhar closed 6 years ago

DhritiShikhar commented 6 years ago

Do not merge. This is a WIP.

fixes #2177

alien-ike commented 6 years ago

Ike Plugins (test-keeper)

Thank you @DhritiShikhar for this contribution!

It seems that this PR already contains some added or changed tests. Good job!

For more information please head over to official documentation. You can find there how to configure the plugin.

codecov[bot] commented 6 years ago

Codecov Report

Merging #2293 into master will decrease coverage by 0.47%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #2293      +/-   ##
=========================================
- Coverage   70.18%   69.7%   -0.48%     
=========================================
  Files         175     175              
  Lines       16844   16696     -148     
=========================================
- Hits        11822   11638     -184     
- Misses       3890    3939      +49     
+ Partials     1132    1119      -13
Impacted Files Coverage Δ
migration/migration.go 61.81% <100%> (-7.48%) :arrow_down:
workitem/workitem_storage.go 73.07% <0%> (-8.51%) :arrow_down:
spacetemplate/template.go 93.1% <0%> (-6.9%) :arrow_down:
space/space.go 64.51% <0%> (-5.14%) :arrow_down:
workitem/link/type_repository.go 61.02% <0%> (-5.1%) :arrow_down:
workitem/typegroup.go 31.52% <0%> (-4.2%) :arrow_down:
workitem/board.go 74.46% <0%> (-3.31%) :arrow_down:
spacetemplate/importer/import_helper.go 54.91% <0%> (-2.35%) :arrow_down:
workitem/json_storage.go 60.6% <0%> (-2.26%) :arrow_down:
... and 27 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 8674b7b...c1d528f. Read the comment docs.

kwk commented 6 years ago

@DhritiShikhar so far this looks like a good start. You might want to write a migration test to proof that it works.

DhritiShikhar commented 6 years ago

Closing this pull request. Will raise another PR for the same.