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

Use maskless bridged jumpers #53

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

cincodenada
Copy link
Contributor

@cincodenada cincodenada commented May 9, 2022

I created maskless versions of the bridged jumpers per discussion on #30 and used them here. I've submitted them upstream to the standard footprint library, so if that gets merged quickly, I'll this PR with any changes from that one.

While I was at made the rotation of the jumpers and naming of the nets consistent, since I noticed there was one inconsistent with the rest. I also made the "(only caps lock key)" text slightly smaller to fit within the lines, since the overlapping silk screen was causing a DRC error in the latest KiCad version.

I also renamed the jumper nets to reference the net they're bridging to (or not) which will hopefully make it more apparent that warnings from tools like Aisler (again, see #30) about shorts are false positives. Is it worth adding a note to the README about this? Also I'm happy to take suggestions on better net names, naming things is my least favorite thing!

I set this as a draft so that we have time to see if the footprints will get merged in, and to facilitate discussion - once things are clearer I'll squash down to one commit.

@cincodenada cincodenada marked this pull request as draft May 9, 2022 05:08
These aren't in the standard library yet but I opened a PR to add them
@cincodenada cincodenada force-pushed the clean-up-jumpers branch 2 times, most recently from 63d4727 to 56c9774 Compare May 9, 2022 05:11
Hopefully this will make things clearer, especially tools give false
positives about shorts
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.

1 participant