Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Request 1209/3320 for MJS2020 sensor board#565

Merged
tannewt merged 1 commit into
pidcodes:masterpidcodes/pidcodes.github.com:masterfrom
meetjestad:mjs2020meetjestad/pidcodes.github.com:mjs2020Copy head branch name to clipboard
Oct 26, 2020
Merged

Request 1209/3320 for MJS2020 sensor board#565
tannewt merged 1 commit into
pidcodes:masterpidcodes/pidcodes.github.com:masterfrom
meetjestad:mjs2020meetjestad/pidcodes.github.com:mjs2020Copy head branch name to clipboard

Conversation

@matthijskooijman

Copy link
Copy Markdown
Contributor

This board is still in development, but the latest prototype version is already published with full sources, the final board will be published in the same place. The same USB id will be used for the prototype and final versions.

@tannewt tannewt mentioned this pull request Oct 14, 2020
50 tasks

@tannewt tannewt left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please pick a PID that doesn't start with 0. They are reserved: http://pid.codes/1209/

@matthijskooijman matthijskooijman changed the title Request 1209/0721 for MJS2020 sensor board Request 1209/3320 for MJS2020 sensor board Oct 20, 2020
@matthijskooijman

Copy link
Copy Markdown
Contributor Author

W00ps. I checked carefully and then decided to change the PID just before submitting the PR, should have rechecked carefully.

I've rebased on top of master and changed the PID to a non-reserved value.

@matthijskooijman

Copy link
Copy Markdown
Contributor Author

I was also planning to submit a PID checker script with github integration, but just noticed that it was already added in the recent merging spree, great! I have a small clarification for the docs, though, which I'll submit in a separate PR.

@tannewt tannewt left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good! Thanks!

@tannewt tannewt merged commit 79c7e02 into pidcodes:master Oct 26, 2020
@matthijskooijman

Copy link
Copy Markdown
Contributor Author

Thanks!

@matthijskooijman matthijskooijman deleted the mjs2020 branch October 29, 2020 15:36
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.