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

updates retrieve_feature_id to ignore case #313

Merged
merged 10 commits into from
Jun 8, 2021
Merged

Conversation

azneto
Copy link
Collaborator

@azneto azneto commented Jun 7, 2021

This pull request addresses issue #...

I hereby agree to licence this and any previous contributions under
the terms of the GNU General Public License version 3 as published by
the Free Software Foundation

I have read the CONTRIBUTING.rst file and understand that
TravisCI will be used to confirm the tests and flake8 style
checks pass with these changes.

I am happy be thanked by name in the CONTRIB.rst files,
and have added myself to the file as part of this pull request.
(This acknowledgement is optional. Note we list the names sorted alphabetically.)

@codecov-commenter
Copy link

codecov-commenter commented Jun 8, 2021

Codecov Report

Merging #313 (55b5084) into master (d94869c) will decrease coverage by 0.03%.
The diff coverage is 50.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #313      +/-   ##
==========================================
- Coverage   69.21%   69.17%   -0.04%     
==========================================
  Files          30       30              
  Lines        4028     4033       +5     
  Branches      228      229       +1     
==========================================
+ Hits         2788     2790       +2     
- Misses       1182     1184       +2     
- Partials       58       59       +1     
Impacted Files Coverage Δ
machado/loaders/common.py 69.47% <50.00%> (-1.64%) ⬇️

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 d94869c...55b5084. Read the comment docs.

@azneto azneto merged commit 7a99773 into lmb-embrapa:master Jun 8, 2021
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.

2 participants