ADIOS2-Examples icon indicating copy to clipboard operation
ADIOS2-Examples copied to clipboard

python 3 syntax

Open dmitry-ganyushin opened this issue 5 years ago • 9 comments

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

dmitry-ganyushin avatar May 01 '20 20:05 dmitry-ganyushin

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

williamfgc avatar May 01 '20 21:05 williamfgc

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

dmitry-ganyushin avatar May 03 '20 18:05 dmitry-ganyushin

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

williamfgc avatar May 07 '20 16:05 williamfgc

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

dmitry-ganyushin avatar May 07 '20 16:05 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.

dmitry-ganyushin avatar May 08 '20 19:05 dmitry-ganyushin

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

williamfgc avatar May 08 '20 19:05 williamfgc

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

pnorbert avatar May 08 '20 21:05 pnorbert

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 avatar May 08 '20 22:05 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.

chuckatkins avatar May 08 '20 22:05 chuckatkins