Update '.allspice/pull_request_template.md'

main
kyle 2 years ago
parent 3ad20ec9c2
commit 5ebc7649c3
  1. 12
      .allspice/pull_request_template.md

@ -26,7 +26,7 @@ about: "Optional description"
- [ ] Power
- [ ] Sufficient power supplied from upstream source
- [ ] Supply rated for necessary country specifications
- [ ] Estimate total worst-case power supply draw
- [ ] Estimated total worst-case power supply draw
- [ ] Connectors
- [ ] I/Os are specified
- [ ] Sufficient Current and Voltage rating
@ -40,7 +40,7 @@ about: "Optional description"
- [ ] Specified min/max humidity
- [ ] ROHS compliance requirement review
### Components
- [ ] Uneccessary components are denoted DNI
- [ ] Unpopulated components are denoted DNI
- [ ] Components meet environmental specifications
- [ ] All components have quantity, reference designator and description
- [ ] Suggested and alternate manufacturers listed
@ -54,7 +54,6 @@ about: "Optional description"
- [ ] Document
- [ ] Dot on each connection
- [ ] No four-point connections
- [ ] Consistent character size for readability
- [ ] Title block completed for each sheet
- [ ] All components have reference designators and values
- [ ] Multi-part components don't have unplaced symbols
@ -88,15 +87,15 @@ about: "Optional description"
- [ ] General
- [ ] Sufficient bulk capacitance calculated
- [ ] Polarized components checked
- [ ] Electrolytic and tantalum capacitors checked for no reverse voltage
- [ ] Debug resources added by design (leds, serial ports, etc.) even if unpopulated by default
- [ ] Electrolytic/tantalum capacitors checked for no reverse voltage
- [ ] Electrolytic/tantalum capacitors temperature/voltage derating sufficient for MTBF
- [ ] Sufficient capacitance on low dropout voltage regulators
- [ ] Sufficient time delays and slew rates for comparators
- [ ] Sufficient common mode input voltage rating on opamps
- [ ] Check pin numbers of all custom-generated parts
- [ ] Electrolytic/tantalum capacitor temperature/voltage derating sufficient for MTBF
- [ ] Check reverse base-emitter current/voltage on bipolar transistors
- [ ] Power nets use preferred and consistent naming (ex. no `3.3V` vs `+3.3V`)
- [ ] Debug resources added by design (leds, serial ports, etc.) even if unpopulated by default
### PCB
- [ ] Manufacturing
- [ ] PCB manufacturing requirements noted on `fab` layer
@ -173,6 +172,7 @@ about: "Optional description"
- [ ] Via fencing of sensitive RF transission lines done with the proper via spacing (< 1/20 lambda)
- [ ] Option for a shielding can over sensitive circuitry e.g. RF?
- [ ] Copper pour
- [ ] All planes have been poured
- [ ] Planes and pours checked for high-impedance paths
- [ ] No pour between adjacent pins on ICs
- [ ] Traces

Loading…
Cancel
Save