这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@jstucke
Copy link
Collaborator

@jstucke jstucke commented Dec 12, 2018

moved FS metadata plugin to core

@codecov-io
Copy link

codecov-io commented Dec 13, 2018

Codecov Report

Merging #186 into master will increase coverage by 0.14%.
The diff coverage is 99.82%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #186      +/-   ##
==========================================
+ Coverage   95.54%   95.68%   +0.14%     
==========================================
  Files         330      334       +4     
  Lines       16597    17159     +562     
==========================================
+ Hits        15857    16418     +561     
- Misses        740      741       +1
Impacted Files Coverage Δ
..._metadata/test/test_file_system_metadata_routes.py 100% <100%> (ø)
..._metadata/test/test_plugin_file_system_metadata.py 100% <100%> (ø)
...ins/analysis/file_system_metadata/routes/routes.py 100% <100%> (ø)
.../file_system_metadata/code/file_system_metadata.py 99.39% <99.39%> (ø)

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 6b1be1b...2908ac6. Read the comment docs.

Copy link
Contributor

@tbarabosch tbarabosch left a comment

Choose a reason for hiding this comment

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

Looks good but there is a file of an old review with comments that shouldn't be in the repo!

@@ -0,0 +1,49 @@
quotation mix:
Copy link
Contributor

Choose a reason for hiding this comment

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

This file shouldn't be here?!

Copy link
Contributor

@weidenba weidenba left a comment

Choose a reason for hiding this comment

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

Please update the "Change Log" on the about page

@tbarabosch
Copy link
Contributor

Looks fine now.

@tbarabosch tbarabosch merged commit 65b35a6 into master Dec 17, 2018
@jstucke jstucke deleted the fs_metadata_plugin branch June 25, 2020 06:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants