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

Build TinyPilot Debian package and install bundle outside the main branch #1214

Merged
merged 4 commits into from
Nov 30, 2022

Conversation

mtlynch
Copy link
Contributor

@mtlynch mtlynch commented Nov 30, 2022

This change adjusts filtering in CircleCI so that we build the TinyPilot Debian package and install bundle even in non-main repo branches.

Uploading a bundle changes state on our update server, so we should only do that in our main branch. But all the other steps are fine to run in branches because it doesn't affect state outside of CI. It's often useful to build packages or bundles in the branches if we want to test something end-to-end.

@mtlynch mtlynch changed the title Build Debian package and bundle outside the main branch Build Debian package outside the main branch Nov 30, 2022
@mtlynch mtlynch changed the title Build Debian package outside the main branch Build TinyPilot Debian package and install bundle outside the main branch Nov 30, 2022
Copy link
Contributor Author

mtlynch commented Nov 30, 2022

Automated comment from CodeApprove ➜

@jotaen4tinypilot please review this Pull Request

@mtlynch mtlynch marked this pull request as ready for review November 30, 2022 14:21
Copy link
Contributor

@jotaen4tinypilot jotaen4tinypilot left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove ➜

Approved on CodeApprove
✔️ Approved

Is it possible to download the bundles from CircleCI?


👀 @mtlynch it's your turn please take a look

Copy link
Contributor Author

@mtlynch mtlynch left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove ➜

In: Discussion
Oh, that's a good idea. Added in #1216.

@mtlynch mtlynch merged commit 15a6514 into master Nov 30, 2022
@mtlynch mtlynch deleted the build-debian-packages branch November 30, 2022 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants