Skip to content

Battery Board - #310

Open
chethankarandikar wants to merge 52 commits into
mainfrom
battery-board
Open

chethankarandikar wants to merge 52 commits into
mainfrom
battery-board

Conversation

@chethankarandikar

Copy link
Copy Markdown
Contributor

No description provided.

@chethankarandikar chethankarandikar left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good to merge i think

@EDED2314 EDED2314 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@KacperParaniuk KacperParaniuk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! lgtm

@tjmcmanamen38 tjmcmanamen38 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Symbol library names should be changed to include manufacturer, e.g. along the lines of:

  • Battery_Charging_Texas or Battery_Texas or Power_Texas or something
  • Display_DisplayVision
  • Regulator_Analog (yes technically its from Linear Technologies but they were absorbed by Analog Devices so I'd still call it Analog lol)

@chethankarandikar

Copy link
Copy Markdown
Contributor Author

renamed symbol libs, moved HUSB238 out of battery charging to its own lib

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.

9 participants