ryansurf / cli-surf

Get surf and ocean data from the command line interface
MIT License
18 stars 30 forks source link

Add windows-platform testing on GitHub Actions CI #74

Closed K-dash closed 4 months ago

K-dash commented 4 months ago

Please Review! @ryansurf (It's incredibly convenient that sourcery-ai reads the diff and generates the PR description for us!)

Summary by Sourcery

This pull request enhances the GitHub Actions CI workflow by adding support for testing on the Windows platform in addition to the existing Ubuntu platform. It introduces a matrix strategy to run tests on both operating systems, ensures the environment file is prepared correctly, and sets the default shell to bash.

Fixed https://github.com/ryansurf/cli-surf/issues/73

sourcery-ai[bot] commented 4 months ago

Reviewer's Guide by Sourcery

This pull request adds support for running tests on both Ubuntu and Windows platforms using GitHub Actions CI. The changes involve modifying the existing CI workflow to include a matrix strategy for different operating systems, preparing the environment file, and ensuring compatibility with both platforms.

File-Level Changes

Files Changes
.github/workflows/pytest.yml Enhanced the CI workflow to support testing on both Ubuntu and Windows platforms, including environment setup and conditional steps.

Tips - Trigger a new Sourcery review by commenting `@sourcery-ai review` on the pull request. - Continue your discussion with Sourcery by replying directly to review comments. - You can change your review settings at any time by accessing your [dashboard](https://app.sourcery.ai): - Enable or disable the Sourcery-generated pull request summary or reviewer's guide; - Change the review language; - You can always [contact us](mailto:support@sourcery.ai) if you have any questions or feedback.