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

allow ALIAS or $FPLIST to be in either order #3

Closed
wants to merge 1 commit into from

Conversation

wez
Copy link

@wez wez commented Sep 18, 2018

The Switch.lib shipping with current versions of kicad
has some of these in a different order from the expectations
of this crate, so add a simple loop that allows them to be
in either order relative to each other.

The Switch.lib shipping with current versions of kicad
has some of these in a different order from the expectations
of this crate, so add a simple loop that allows them to be
in either order relative to each other.
@adamheinrich adamheinrich self-assigned this Oct 7, 2018
@adamheinrich
Copy link
Member

adamheinrich commented Oct 8, 2018

Hi @wez, thank your for your PR and sorry for the delay. Could you please make the failing test work?

@wez
Copy link
Author

wez commented May 12, 2019

The tests do pass; it looks like travis had a blip and flaked out on its own. You should be able to restart the job; I can't do anything about it because I don't own this project, and I can't rebase and re-submit because nothing has changed in this repo since I authored this PR

@adamheinrich
Copy link
Member

Merged in #5, thanks.

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