quick-camera icon indicating copy to clipboard operation
quick-camera copied to clipboard

Continuous Integration

Open ajmas opened this issue 4 years ago • 2 comments

Now that GitHub supports Continuous Integration, via GitHub workflows, it could be interesting to have this project have a CI target for this, even if the builds aren't signed.

ajmas avatar Jan 24 '21 12:01 ajmas

This is a great idea. The challenge I've faced (other than lack of time to look into this!) is how to include a test suite with any CI implementation. i.e. how to prove no regression for mirroring/rotating the image. I guess one way could be to mock/stub out the AVCaptureDevice, but that's likely a non-trivial piece of work.

simonguest avatar Jan 30 '21 20:01 simonguest

While tests would be ideal, maybe simply start with validating the app build and run the code through a linter to ensure contributions respect a prescribed code style. For example, whether to require semi-colons or not.

ajmas avatar Jan 31 '21 11:01 ajmas