python 3 syntax
in gsplot.py
raise "abc"
probably is not a valid python 3 syntax
@dmitry-ganyushin welcome! Please feel free to start contributing. I don't have a CI for Python3 components. It just verifies that C and C++ executables build and run.
Thanks, William. I will clean-up python files and check them with pylint.
@dmitry-ganyushin thanks! Do you think there is a way to put pylint on CI? The build.sh is run on azure just to make sure the library builds and installs.
Sure, there is a way to check it in CI. Another possibility is to check it before commit. What would you prefer?
There is a way to define an action on github which should check python files with pylint after every push. You do not have to wait for a big build on azure just to check python syntax.
I'd check with @pnorbert , I just added azure-pipelines as it predates GitHub Actions. Feel free to add what makes more sense.
@chuckatkins is using both (and travis) in the adios2 repository. So use whichever you think is better.
Formatting checks are run on Travis with clang-format for the c++ code and flake8 for the Python code and is executed immediately on push. We can certainly add pylint to that.
In the adios2 repo we use a combination of Travis, circle, azure pipelines, and GitHub actions, each for different use cases, in order to maximize our throughout and not get ci backlogs during times of high activity.