Skip to content

Conversation

@knapii-developments
Copy link
Contributor

@knapii-developments knapii-developments commented Sep 18, 2020

1889670378- We're reporting when cell metadata format is valid with logger.error when it should be reported with logger.info. Because it's reported with logger.error it's being reported in sentry when it shouldn't.

@codecov
Copy link

codecov bot commented Sep 18, 2020

Codecov Report

Merging #147 into development will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##           development     #147   +/-   ##
============================================
  Coverage        69.00%   69.00%           
============================================
  Files               22       22           
  Lines             2678     2678           
============================================
  Hits              1848     1848           
  Misses             830      830           
Impacted Files Coverage Δ
ingest/ingest_pipeline.py 54.22% <100.00%> (ø)

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 6d55525...b3f9736. Read the comment docs.

@knapii-developments knapii-developments merged commit 26f2a5e into development Sep 21, 2020
@knapii-developments knapii-developments deleted the ea-fix-cell-metadata-logger branch September 21, 2020 13:49
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