Skip to content
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

Add Python code quality scanning #727

Closed
wants to merge 6 commits into from
Closed

Add Python code quality scanning #727

wants to merge 6 commits into from

Conversation

sambles
Copy link
Contributor

@sambles sambles commented Jan 24, 2023

Add Python code quality scanning

Warning: This pull request depends on having #724 merged into develop first

@sambles sambles requested a review from mtazzari January 24, 2023 13:47
@sambles sambles marked this pull request as draft January 24, 2023 13:47
@sambles sambles linked an issue Jan 24, 2023 that may be closed by this pull request
test file return format conv

This reverts commit 8c0de7e.

Draft option to validate oed files on upload

validate on upload - wip

Note for later

Draft serializer to return portolio validation status

WIP GET validated files

POST portfolio validate

Add validation url param

read validation option, url/settings.py

Set ods-tools 3.0.1

fix unittests

Fix validation on parquet file uploads

Fix handling of invalid data upload

Fix typos and missing docs strings

remove dup func

Turn valadation on by default

Add validation unit testing

Add test_all_exposure__are_valid
@sambles sambles added Enhancement Small improvement or refinement. build system port forward labels Jan 26, 2023
@sambles
Copy link
Contributor Author

sambles commented Jan 30, 2023

Moved to #732

@sambles sambles closed this Jan 30, 2023
@sambles sambles deleted the actions/code-ql branch January 30, 2023 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build system Enhancement Small improvement or refinement.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Add Code QL to platform repo
1 participant