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

Update v2 #19

Merged
merged 992 commits into from
Nov 16, 2021
Merged

Update v2 #19

merged 992 commits into from
Nov 16, 2021

Conversation

jspeed-meyers
Copy link
Contributor

@rob, this is an attempt to merge back in the progress from the private fork into the public fork. Does it look right to you? Can you please take a look and let me know?

JS

mchadwick-iqt and others added 30 commits June 3, 2021 09:48
Former-commit-id: e43ed6987551bce997ce1b03fe80c0d9c52f1d40
Former-commit-id: 349ca1bf49a90e9d7e0185e70d5d5f0ffed18e6b
Former-commit-id: 52cb00d34eb708b9acb01831d0be0c0190bdfde5
Former-commit-id: 36e66b29f3c1978214e7448d1d80b435c6f11be2
Former-commit-id: e3c111128a4ec28cea44fa328adece47250b6a71
Former-commit-id: 042131c58d48157ea989b37802949833aff6fbd3
Former-commit-id: 92cda3b7c976cf87f898cd33ae25be6ba52887c9
Former-commit-id: 72bc96c5811125e038afebea7c39fc66a518fd06
Former-commit-id: 81bf06e9296415a8e47a69844cb3a9f965a62f41
Former-commit-id: 6fd5abfdbfc05b46b98fd3c441c980265178c020
Former-commit-id: 2980460f292d8f1e515583ffda69d1eeeb4520e0
Former-commit-id: 8e030db2cc926f5a5bc2b0186b78e7f309238fbc
Former-commit-id: f1dd4b5bcdc6f5f63db4ff5fa3e16e0f0799b35c
Former-commit-id: e00f44353c011245ac2e2fc2c98cf0fa91f5436a
Former-commit-id: 9901dfe83a99d37f110efc35673a23082831791d
@jspeed-meyers jspeed-meyers added documentation Improvements or additions to documentation enhancement New feature or request labels Nov 15, 2021
@jspeed-meyers jspeed-meyers self-assigned this Nov 15, 2021
@jspeed-meyers
Copy link
Contributor Author

@mchadwick-iqt, does this look right to you? Of course, this operation is not as simple as I would like it to be.

Copy link
Member

@rcaudill rcaudill left a comment

Choose a reason for hiding this comment

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

@jspeed-meyers this looks correct to me!

@mchadwick-iqt
Copy link
Contributor

@jspeed-meyers - I pushed a new commit to remove a number of obsolete files that had been retained with the merge. Can you do a quick once-over and make sure we're still good from your perspective?

@jspeed-meyers
Copy link
Contributor Author

@mchadwick-iqt, It looks good to me. I am going to look at it one more time with fresh eyes tomorrow morning and then I'll merge.

@@ -4,5 +4,5 @@ services:
axis-ptz:
command: /bin/sh -c "while sleep 1000; do :; done"
volumes:
- /home/pi/Projects/SkyScan/axis-ptz:/workspace:cached # Additional bind mount
- /home/pi/Projects/SkyScan-Private/axis-ptz:/workspace:cached # Additional bind mount
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@rcaudill, is this use of a different volume path a problem?

@mchadwick-iqt mchadwick-iqt merged commit 4cf7816 into main Nov 16, 2021
mchadwick-iqt pushed a commit that referenced this pull request Nov 16, 2021
Web status

Former-commit-id: dbf96fb6813361d883c76a7fe11203503e92556c
mchadwick-iqt pushed a commit that referenced this pull request Nov 16, 2021
@mchadwick-iqt mchadwick-iqt deleted the update_v2 branch February 23, 2023 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants