search

catalystneuro / buzsaki-lab-to-nwb

BSD 3-Clause "New" or "Revised" License
1 stars 1 forks source link

Add pre-commit to Buzsaki #62

Closed h-mayorquin closed 1 year ago

h-mayorquin commented 1 year ago

I added the pre-commit and the channel map thing we did today.

What do you think, @CodyCBakerPhD ?

CodyCBakerPhD commented 1 year ago

Looks good, but might need to add the isort section to the pyproject.toml

Taken from NeuroConv,

[tool.isort]
profile = "black"
reverse_relative = true
known_first_party = ["buzsaki_lab_to_nwb"]
CodyCBakerPhD commented 1 year ago

So what workflow worked best for SI? To enable the bot after this is merged into main and after PRs are updated from that?

h-mayorquin commented 1 year ago

Added isort. I think that workflow will work here.

h-mayorquin commented 1 year ago

I also updated the install action.