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.

Sign up for GitHub

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

Jump to bottom

Consolidate the ansible-role-ustreamer repo into the TinyPilot repo #1426

Closed
jdeanwallace opened this issue Jun 6, 2023 · 0 comments · Fixed by #1434
Closed

Consolidate the ansible-role-ustreamer repo into the TinyPilot repo #1426

jdeanwallace opened this issue Jun 6, 2023 · 0 comments · Fixed by #1434
Assignees
@jdeanwallace
Labels
small

Comments

@jdeanwallace
Copy link
Contributor

Related #1353
Blocked on tiny-pilot/ansible-role-ustreamer#113

Seeing as this is just a temporary solution we can simply move the contents of the ansible-role-ustreamer repo into a ansible-role-ustreamer directory, at the root of the TinyPilot repo.
@jdeanwallace jdeanwallace transferred this issue from tiny-pilot/ansible-role-ustreamer Jun 6, 2023
This was referenced Jun 6, 2023
Closed
Closed
Closed
@jdeanwallace jdeanwallace self-assigned this Jun 12, 2023
This was referenced Jun 12, 2023
Merged
Merged
Merged
jdeanwallace added a commit that referenced this issue Jun 12, 2023
@jdeanwallace
Fix shellcheck issues in ansible-role-ustreamer (#1436)
3845e12 
Related #1426
Dependent on #1434

Caveat: This change is just to satisfy the [`check_bash` CI
check](https://app.circleci.com/pipelines/github/tiny-pilot/tinypilot/3462/workflows/cd3bda98-6f98-4a4c-b860-0222cb4cf272/jobs/19191).
We'll soon remove the
`ansible-role-ustreamer/tests/check-trailing-newline` script entirely.

<a data-ca-tag
href="https://codeapprove.com/pr/tiny-pilot/tinypilot/1436"><img
src="https://codeapprove.com/external/github-tag-allbg.png" alt="Review
on CodeApprove" /></a>
jdeanwallace added a commit that referenced this issue Jun 12, 2023
@jdeanwallace
Fix code style in ansible-role-ustreamer (#1435)
0fce6c4 
Related #1426
Dependent on #1434

This PR is the result of running
[`dev-scripts/fix-style`](dev-scripts/fix-style) on the code in
`ansible-role-ustreamer/`.

<a data-ca-tag
href="https://codeapprove.com/pr/tiny-pilot/tinypilot/1435"><img
src="https://codeapprove.com/external/github-tag-allbg.png" alt="Review
on CodeApprove" /></a>
jdeanwallace added a commit that referenced this issue Jun 15, 2023
@jdeanwallace
Consolidate the ansible-role-ustreamer repo into the TinyPilot repo (#…
7a067d3 
…1434)

Resolves #1426
Blocked on
tiny-pilot/ansible-role-ustreamer#113

This PR is an exact copy of the [`ansible-role-ustreamer`
repo](https://github.com/tiny-pilot/ansible-role-ustreamer/tree/02f37b869c8c585517a9c7b6cd95cfb7661f1490)
into the `ansible-role-ustreamer/` directory.

The CI checks are resolved in subsequent PRs:
* #1435
* #1436

<a data-ca-tag
href="https://codeapprove.com/pr/tiny-pilot/tinypilot/1434"><img
src="https://codeapprove.com/external/github-tag-allbg.png" alt="Review
on CodeApprove" /></a>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jdeanwallace jdeanwallace

Labels
small
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Consolidate the ansible-role-ustreamer repo into the TinyPilot repo tiny-pilot/tinypilot
1 participant
@jdeanwallace