You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
IMO the preamble should only feature the authors and the copyright note. This is similar to what mne-python does (see https://github.com/mne-tools/mne-python/blob/main/mne/decoding/time_delaying_ridge.py) and feels much easier to maintain. I think keeping the authors in cpp and h files is important because this is an open-source project after all. The people who mainly worked on these files should be named here.
Next to the preamble, I also changed the following:
Removed all /==== separators as I agree with @juangpc that they do not add much value
Removed all docs blocks such as Qt includes, inlines, etc.
Based on a discussion in #902 with @gabrielbmotta and @juangpc.
IMO the preamble should only feature the authors and the copyright note. This is similar to what mne-python does (see https://github.com/mne-tools/mne-python/blob/main/mne/decoding/time_delaying_ridge.py) and feels much easier to maintain. I think keeping the authors in cpp and h files is important because this is an open-source project after all. The people who mainly worked on these files should be named here.
Next to the preamble, I also changed the following:
/====
separators as I agree with @juangpc that they do not add much value//!
for single line commentsclang-format -style=LLVM -I connectivity.h
, see Introduce tool for consistent formatting #905 for detailed discussion on clang-formatAn example (libraries/connectivity/connectivity.h):
The text was updated successfully, but these errors were encountered: