Combined fixes from reviews
Summary
Addresses improvements suggested in the reviews of https://gitlab.indiscale.com/caosdb/customers/gwdg/management/-/issues/198 and https://gitlab.indiscale.com/caosdb/customers/gwdg/management/-/issues/202 as well as the enhancements proposed in https://gitlab.indiscale.com/caosdb/customers/gwdg/management/-/issues/216.
- Added a bash script to auto-build the python client
- Expanded checks with handle system
- localhost can be configured with env variable
- Added a .gitignore
- Refactors + Style Fixes:
- Reworked utils, split off assertions and refactored the methods to standardize usage
- Added API versions to config
- file paths are now built independent of folder names
- Removed static methods, example files, .keys()
- Improved readability
- Ran autopep8
- Fixed all pylint errors, warnings, and convention warnings or added ignores where a fix was not feasible
Testing
Testing the bash script on another system is necessary. No setup should be needed.
Check List for the Author
-
All automated tests pass -
Reference related issues -
Appropriate user and developer documentation for new code -
Annotations in code (Gitlab comments)
Check List for the Reviewer
-
I understand the intent of this MR -
All automated tests pass -
Appropriate user and developer documentation for new code -
The test environment setup works and the intended behavior is reproducible -
In-code documentation and comments are up-to-date. -
The bash script has been tested
Edited by I. Nüske