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

Fixes some lax parsing tests #6

Closed
wants to merge 4 commits into from
Closed

Conversation

dvc94ch
Copy link
Collaborator

@dvc94ch dvc94ch commented Jan 23, 2018

No description provided.

Renames characteristic -> dielectric.
Renames size -> package_size.
Only uses reasonable units (1F -> 1uF).
Uncomments tests that fail because of tolerance position.
@kasbah
Copy link
Member

kasbah commented Jan 23, 2018

Thanks! Merged this into the lax-parser branch and closing this in favor of #8 where I will continue work.

@kasbah kasbah closed this Jan 23, 2018
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