Skip to content
This repository has been archived by the owner on Oct 2, 2020. It is now read-only.

Try to fix #955 #2627

Closed
wants to merge 4 commits into from
Closed

Try to fix #955 #2627

wants to merge 4 commits into from

Conversation

aewallin
Copy link
Contributor

mostly footprint-filter changes.

The Si7050 symbol did not have an existing footprint assigned.
https://www.silabs.com/documents/public/data-sheets/Si7050-1-3-4-5-A20.pdf

@aewallin
Copy link
Contributor Author

ping #955

@myfreescalewebpage myfreescalewebpage added Bug Fix symbol existing in the library Pending reviewer A pull request waiting for a reviewer labels Apr 13, 2020
@myfreescalewebpage myfreescalewebpage self-assigned this Apr 13, 2020
@myfreescalewebpage myfreescalewebpage removed the Pending reviewer A pull request waiting for a reviewer label Apr 13, 2020
@myfreescalewebpage
Copy link
Collaborator

myfreescalewebpage commented Apr 13, 2020

Hello @aewallin

Thanks for fixing the issues.

Some comments:

  • For KTY85 device, can you fix the name of the package in the description as well ?I think "Silicon temperature sensor, polarized, MiniMELF" is great
  • For SC-70 filters, you can use SOT?353*

If you agree there are also some Pin Offset warning reported by travis on the same devices, if you have time to fix them.

Joel

@aewallin
Copy link
Contributor Author

I suggest the librarians describe exactly the work desired in #955 in sufficient detail.
closing this for now - possibly to revisit if instructions in issues are clear enough.

@aewallin aewallin closed this May 10, 2020
@myfreescalewebpage myfreescalewebpage added the Abandoned Original author has stopped working on the PR label May 10, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Abandoned Original author has stopped working on the PR Bug Fix symbol existing in the library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants