nlesc-recruit / cudawrappers

C++ wrapper for the Nvidia C libraries (e.g. CUDA driver, nvrtc, cuFFT etc.)
https://cudawrappers.readthedocs.io/en/latest/
Apache License 2.0
5 stars 5 forks source link

Add requirements.txt with linters/formatters that can be installed from PyPI #90

Closed abelsiqueira closed 2 years ago

abelsiqueira commented 2 years ago

We could put the things you can get from pypi into a requirements.txt

_Originally posted by @jspaaks in https://github.com/nlesc-recruit/CUDA-wrappers/pull/77#discussion_r816741332_

fdiblen commented 2 years ago

Also update the developer documentation.

jspaaks commented 2 years ago

Alternatively, maybe we can just add a note to the README.dev.md, something along th elines of

# install cppcheck, clang-tidy, clang-format from your package manager
sudo apt install cppcheck clang-format clang-tidy

# install python3 and pip
sudo apt install python3 python3-pip

# install flawfinder, cmake-format, cmake-lint from PyPI using pip
python3 -m pip install flawfinder cmake-format cmake-lint 
fdiblen commented 2 years ago

Alternatively, maybe we can just add a note to the README.dev.md, something along th elines of

# install cppcheck, clang-tidy, clang-format from your package manager
sudo apt install cppcheck clang-format clang-tidy

# install python3 and pip
sudo apt install python3 python3-pip

# install flawfinder, cmake-format, cmake-lint from PyPI using pip
python3 -m pip install flawfinder cmake-format cmake-lint 

I think this will be cleaner so yes!

fdiblen commented 2 years ago

fixed by #105