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

V2 #46

Merged
merged 47 commits into from
Apr 6, 2021
Merged

V2 #46

merged 47 commits into from
Apr 6, 2021

Conversation

phil-d-wilson
Copy link
Contributor

No description provided.

Change-type: major
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: patch
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: patch
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: patch
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: minor
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: minor
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: patch
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: patch
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: patch
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: minor
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: patch
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: minor
Signed-off-by: Phil Wilson <phil@balena.io>
.gitignore Outdated Show resolved Hide resolved
phil-d-wilson and others added 2 commits January 29, 2021 11:53
Change-type: patch
Signed-off-by: Phil Wilson <phil@balena.io>
Copy link
Contributor

@tmigone tmigone left a comment

Choose a reason for hiding this comment

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

👯‍♂️


# Set up the audio block. This won't have any effect if the audio block is not being used.
RUN curl -skL https://raw.githubusercontent.com/balenablocks/audio/master/scripts/alsa-bridge/debian-setup.sh| sh
ENV PULSE_SERVER=tcp:audio:4317
Copy link
Contributor

Choose a reason for hiding this comment

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

Did you test the integration with the audio block? If not let me know and I can do it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@tmigone I haven't yet. If you wouldn't mind testing the block in general, and the audio link specifically, that would be much appreciated.

src/server.js Outdated Show resolved Hide resolved
src/server.js Outdated Show resolved Hide resolved
src/server.js Outdated Show resolved Hide resolved
src/server.js Outdated Show resolved Hide resolved
src/server.js Outdated Show resolved Hide resolved
src/server.js Outdated

flags = [];
// If the user has set the flags, use them
if (null != FLAGS)
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure if there is any difference but I've almost never seen the variable on the right side of a comparison in js. It probably works exactly in the same way, so most likely a stylistic thing.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@tmigone you're right - it's poor practice in any language, since accidentally typing '=' rather than '==' or '===' can create odd bugs which are hard to track down. It's my sloppiness coming out just because I'm hacking something in a language I don't fully get. :)

src/server.js Outdated Show resolved Hide resolved
src/server.js Outdated Show resolved Hide resolved
src/server.js Show resolved Hide resolved
Change-type: patch
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: patch
Signed-off-by: Phil Wilson <phil@balena.io>
@phil-d-wilson phil-d-wilson marked this pull request as ready for review February 2, 2021 17:55
phil-d-wilson and others added 5 commits February 2, 2021 17:56
Change-type: patch
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: patch
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: patch
Signed-off-by: Tomás Migone <tomas@balena.io>
tmigone and others added 25 commits March 1, 2021 10:55
Change-type: patch
Signed-off-by: Tomás Migone <tomas@balena.io>
meta: run versionist to update balena.yml version
Change-type: major
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: patch
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: patch
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: patch
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: minor
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: minor
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: patch
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: patch
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: patch
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: minor
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: patch
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: minor
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: patch
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: patch
Signed-off-by: Phil Wilson <phil@balena.io>
Change-type: patch
Signed-off-by: Phil Wilson <phil@balena.io>
@phil-d-wilson phil-d-wilson merged commit 2cda06b into master Apr 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants