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

fix(shared-data): fix "strict" arg for labware creation #3874

Merged
merged 1 commit into from
Aug 16, 2019

Conversation

IanLondon
Copy link
Contributor

overview

I messed up #3804, if strict was false it would "default" to true because of the ||, oops!

changelog

review requests

Code review, see unit tests.

@IanLondon IanLondon added fix PR fixes a bug shared data Affects the `shared-data` project labels Aug 16, 2019
Copy link
Contributor

@mcous mcous left a comment

Choose a reason for hiding this comment

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

📏

@codecov
Copy link

codecov bot commented Aug 16, 2019

Codecov Report

Merging #3874 into edge will increase coverage by 0.03%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##             edge    #3874      +/-   ##
==========================================
+ Coverage   34.61%   34.65%   +0.03%     
==========================================
  Files         731      731              
  Lines       11630    11630              
==========================================
+ Hits         4026     4030       +4     
+ Misses       7604     7600       -4
Impacted Files Coverage Δ
shared-data/js/labwareTools/index.js 92.1% <100%> (+5.26%) ⬆️

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 9ab6938...ca0d938. Read the comment docs.

@IanLondon IanLondon merged commit bd604e2 into edge Aug 16, 2019
@IanLondon IanLondon deleted the shared-data_fix_strict-arg branch August 16, 2019 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix PR fixes a bug shared data Affects the `shared-data` project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants