Skip to content

Conversation

@rodrigoalmeida94
Copy link
Collaborator

@rodrigoalmeida94 rodrigoalmeida94 commented Feb 1, 2022

  • Fix linter errors
  • Adds CI/CD for testing
  • Adds CI/CD for release

SRM and others added 6 commits January 31, 2022 12:14
- Remove python-env from tox.ini files, use gh-actions matrix for
running different python versions
- Define tox runs as `tox -e [lint/format/flake8/test]`
- Tested github-actions locally using act.
- osgeo ubuntu-large images are 1.4 GB in size, takes some time to run this locally
- Replacing this with ubuntu-small runs it faster with no side effects
* Uncomment tox

* Update job name to test

* Add release workflow only on main and with tag
* lint is flake8

* Remove - in order to not ignore exit code

* Add extended ignores for flake8

* Rm unused vars

* Remove extra #

* Rm unused var

* Add rio-cogeo dependency

* Fix undefined variable, ignore ambigous name

* Rm flake8

* Fix unused imports F401
@rodrigoalmeida94 rodrigoalmeida94 changed the base branch from main to 0.5.0 February 1, 2022 12:40
@rodrigoalmeida94 rodrigoalmeida94 marked this pull request as ready for review February 1, 2022 12:43
@rodrigoalmeida94
Copy link
Collaborator Author

@srmsoumya you can close #446 in favour of this PR. Let me know if this looks good!

@rodrigoalmeida94 rodrigoalmeida94 changed the title Add CI/CD github action Add CI/CD in Github Actions Feb 1, 2022
@rodrigoalmeida94 rodrigoalmeida94 force-pushed the feature/440-cicd-ghaction branch from 734bfb8 to 4f0670c Compare February 1, 2022 12:49
@rodrigoalmeida94 rodrigoalmeida94 force-pushed the feature/440-cicd-ghaction branch from 4142e56 to 8412a0d Compare February 8, 2022 11:23
@rodrigoalmeida94 rodrigoalmeida94 merged commit 0dc498b into 0.5.0 Feb 8, 2022
@rodrigoalmeida94 rodrigoalmeida94 deleted the feature/440-cicd-ghaction branch February 8, 2022 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

move all ci/cd to github actions, including testing, linting, and codecov

5 participants