-
Notifications
You must be signed in to change notification settings - Fork 0
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
[PUT Project] Added CI with format testing and building #2
Conversation
ftr, this PR also includes commits from #1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please rebase this PR on top of #1
79997a8
to
c89b732
Compare
rebased onto #1 (+ changed the target branch) ptal @glatosinski |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left some minor requests, but in general the test of formats with vivid looks good
cba76d7
to
2706198
Compare
@glatosinski applied the suggestions, dropped the DNM |
actually the pipeline fails... i was sure that it was green, sorry for spam |
GH Actions: grabthecam-build: Fixed code linting - maybe let's squash it with the first commit that actually introduces this change? |
squashed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This PR adds the full CI implementation for testing image format conversion (taking into account that some formats can be lossy) and building for grabthecam.