fixed suggestions as per ruff #54
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
francbartoli
pushed a commit
that referenced
this pull request
Nov 4, 2024
* addd layer rest api (#43) * added layer REST * updated all pages * param workspace added to get_all_layers * added delete layer api * addd layer rest api (#43) * added layer REST * updated all pages * param workspace added to get_all_layers * added delete layer api * upgraded version in pyproject (#44) * fixed all pydantic calls (#51) * fixed all pydantic calls * upgraded constraints file * updated version of checkout and python in test.yml * updated poetry version in contraints * trying poetry installer * updated poetry installation method in release.yml * fixed few pydanticv2 issues (#53) * fixed suggestions as per review (#54) * Feature/create-stores (#46) * added vector and raster store function * added vector and raster store * fixed test issue * updated add vector layer to omit null * updated good_layer_connection * fixed suggestions as per review * added vector and raster store function * added vector and raster store * fixed test issue * updated add vector layer to omit null * updated good_layer_connection * fixed suggestions as per review * added vector and raster store * updated add vector layer to omit null * Feature/layer-group (#52) * added get layer group function * fixed model names and lowercased function names in test * added tests for LayerGroupsModel * fixed as per pydanticv2 protocol (#55) * fixed issues as per ruff (#57) * using black isort and ruff in the test CI pipeline (#63) * using black isort and ruff in the test CI pipeline * removed duplicate installation * testing with black and isort marketplace options * testing black action * testing with ruff * fixed black . * updated black and ruff version and run black again * fixed as per isort * added profile black to isort in CI * testing if ruff gives error in CI * fixed black but keeping error as per ruff * fixed as per ruff * updated readme (#59) * updated readme * using black isort and ruff in the test CI pipeline (#63) * using black isort and ruff in the test CI pipeline * removed duplicate installation * testing with black and isort marketplace options * testing black action * testing with ruff * fixed black . * updated black and ruff version and run black again * fixed as per isort * added profile black to isort in CI * testing if ruff gives error in CI * fixed black but keeping error as per ruff * fixed as per ruff * Feature/get-geofence (#56) * added base code to get geofence rules * added header to geofence function * added tests for geofence * added single geofence rule * adding new geofence rule * added check_modules as per #58 * linting done * added try: except in checl_module * hotfix response schema to get geofence rule * fixed as black * hotfix - returning response in case of http error * fixed remaining geofence functions * added custom exception * fixed as per black * updated version to 0.0.4 (#64) * updated version to 0.0.4 * updated description * added readme config to toml * update documents (#65) * update docs * updated requirements.txt * updated flow for docs * updating workflow file * added site_url to mkdocs * updated toml for mypu * updated lock file * fixed as per black
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
ruff check . --fix