Closed lcortesg closed 1 month ago
github-actions[bot]
commented
1 month ago 👋 Hi! Thank you for contributing to the EEGraSP project.
Just a reminder: Any PR opened on any branch, other than main will automatically merge once the test-merge.yml test is passed. The other lint test are optional, and will not interfere with the automerge if they fail.
🚀
github-actions[bot]
commented
1 month ago 👋 Hi! Thank you for contributing to the EEGraSP project.
Just a reminder: Any PR opened on any branch, other than main will automatically merge once the test-merge.yml test is passed. The other lint test are optional, and will not interfere with the automerge if they fail.
🍌 🐒 🚀
github-actions[bot]
commented
1 month ago 👋 Hi! Thank you for contributing to the EEGraSP project.
Just a reminder: Any PR opened on any branch, other than main will automatically merge once the test-merge.yml test is passed. The other lint test are optional, and will not interfere with the automerge if they fail.
🍌 🐒 🚀
Note that these only check if the tool is able to automatically lint the code, but it doesn't change anything, for this an auto-linting stage is proposed.
Some of these checks might fail, due to the source code not being properly linted, for that a local lint stage was update in the makefile, once these changes are approved, a new linted version of the code might be merged into main.
The codespell check is failing due to a misspelled word with multiple suggestions (./eegrasp/tests/load_from_mne.py:65: usign ==> using, unsign and ./eegrasp/tests/load_from_mne.py:74: usign ==> using, unsign), this can be addressed in the lint full codebase issue.