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

Add overlays for justboom and reorder #788

Merged
merged 2 commits into from Jan 19, 2021
Merged

Conversation

shawaj
Copy link
Contributor

@shawaj shawaj commented Dec 26, 2020

Add overlays for justboom audio boards and reorder list to be in alphabetical order

Change-type: patch
Signed-off-by: Aaron Shaw aaron@balena.io

@agherzan-ci
Copy link

Can one of the admins verify this patch?

Copy link
Collaborator

@kraj kraj left a comment

Choose a reason for hiding this comment

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

usually adding and reordering should be two different patches

@shawaj
Copy link
Contributor Author

shawaj commented Dec 27, 2020

@kraj fair enough. So just split into 2 different commits? But keeping on a single PR is ok?

@kraj
Copy link
Collaborator

kraj commented Dec 27, 2020

@kraj fair enough. So just split into 2 different commits? But keeping on a single PR is ok?

yes absolutely.

@shawaj shawaj force-pushed the patch-3 branch 3 times, most recently from 60d3264 to 9b8b3bc Compare January 14, 2021 22:24
reorder list to be in alphabetical order

Change-type: patch
Signed-off-by: Aaron Shaw <aaron@balena.io>
add justboom overlays

Change-type: patch
Signed-off-by: Aaron Shaw <shawaj@gmail.com>
@shawaj
Copy link
Contributor Author

shawaj commented Jan 14, 2021

@kraj fixed now :-)

Copy link
Collaborator

@kraj kraj left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Collaborator

@kraj kraj left a comment

Choose a reason for hiding this comment

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

lgtm

@shawaj
Copy link
Contributor Author

shawaj commented Jan 15, 2021

@kraj does the CI need to test build? How do I / we trigger that?

@kraj
Copy link
Collaborator

kraj commented Jan 15, 2021

@kraj does the CI need to test build? How do I / we trigger that?

it should start on pull request refresh or push

@shawaj
Copy link
Contributor Author

shawaj commented Jan 15, 2021

@kraj it doesn't seem to have worked since my last updates?

@shawaj
Copy link
Contributor Author

shawaj commented Jan 15, 2021

@agherzan-ci ok to test

1 similar comment
@agherzan
Copy link
Owner

@agherzan-ci ok to test

@agherzan agherzan merged commit 9165aa5 into agherzan:master Jan 19, 2021
@shawaj shawaj deleted the patch-3 branch January 20, 2021 03:55
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.

None yet

4 participants