Skip to content

Commit

Permalink
Merge pull request #3402 from h-mayorquin/add_header_decision_to_dev_doc
Browse files Browse the repository at this point in the history
Add decision about headers to dev docs
  • Loading branch information
alejoe91 authored Sep 12, 2024
2 parents c176830 + 67469ac commit 8d9f8db
Showing 1 changed file with 9 additions and 1 deletion.
10 changes: 9 additions & 1 deletion doc/development/development.rst
Original file line number Diff line number Diff line change
Expand Up @@ -189,9 +189,17 @@ so that the user knows what the options are.
Miscelleaneous Stylistic Conventions
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

#. Avoid using abreviations in variable names (e.g., use :code:`recording` instead of :code:`rec`). It is especially important to avoid single letter variables.
#. Avoid using abbreviations in variable names (e.g. use :code:`recording` instead of :code:`rec`). It is especially important to avoid single letter variables.
#. Use index as singular and indices for plural following the NumPy convention. Avoid idx or indexes. Plus, id and ids are reserved for identifiers (i.e. channel_ids)
#. We use file_path and folder_path (instead of file_name and folder_name) for clarity.
#. For creating headers to divide sections of code we use the following convention (see issue `#3019 <https://github.com/SpikeInterface/spikeinterface/issues/3019>`_):


.. code:: python
#########################################
# A header
#########################################
How to build the documentation
Expand Down

0 comments on commit 8d9f8db

Please sign in to comment.