python 3 syntax

[dmitry-ganyushin]

in gsplot.py
raise "abc"
probably is not a valid python 3 syntax

[williamfgc]

@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.

[dmitry-ganyushin]

Thanks, William. I will clean-up python files and check them with pylint.

[williamfgc]

@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.

[dmitry-ganyushin]

Sure, there is a way to check it in CI. Another possibility is to check it before commit. What would you prefer?

[dmitry-ganyushin]

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.

[williamfgc]

I'd check with @pnorbert , I just added azure-pipelines as it predates GitHub Actions. Feel free to add what makes more sense.

[pnorbert]

@chuckatkins is using both (and travis) in the adios2 repository. So use whichever you think is better.

[chuckatkins]

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.

[chuckatkins]

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.