Skip to content

Add Alorium Technology Evo M51#481

Merged
makermelissa merged 3 commits into
adafruit:masterfrom
jpecor:evo_m51
May 22, 2020
Merged

Add Alorium Technology Evo M51#481
makermelissa merged 3 commits into
adafruit:masterfrom
jpecor:evo_m51

Conversation

@jpecor
Copy link
Copy Markdown
Contributor

@jpecor jpecor commented May 21, 2020

No description provided.

@dhalbert dhalbert requested a review from makermelissa May 21, 2020 22:50
Copy link
Copy Markdown
Collaborator

@makermelissa makermelissa left a comment

Choose a reason for hiding this comment

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

I just see one change to make, but looks good. Thanks.

Comment thread _board/aloriumtech_evo_m51.md Outdated
date_added: 2020-5-21
downloads_display: true
blinka: false
download_instructions: "BLINKA ONLY - url"
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.

Can we remove this line? It looks like you used the template and for non-Blinka boards it should be removed. Thanks

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.

Yes. I wasn't sure what to do with that line, so I left it in. So, no problem removing it.

Is that something I do?

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.

Yeah, just remove it and push the change.

Copy link
Copy Markdown
Collaborator

@makermelissa makermelissa left a comment

Choose a reason for hiding this comment

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

Looks great! Thank you.

@makermelissa makermelissa merged commit 0b678ca into adafruit:master May 22, 2020
@jpecor jpecor deleted the evo_m51 branch August 28, 2020 18:39
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