Skip to content

Conversation

@hoechenberger
Copy link
Member

@hoechenberger hoechenberger commented Nov 25, 2023

My linter showed the entire pyproject.toml file in red, so I went on the investigate.

Turns out both section and key names were incorrect. I fixed that. I also removed the zip-safe flag because it's been deprecated for many years and isn't used anymore.

BTW the linter I'm using is https://marketplace.visualstudio.com/items?itemName=tamasfe.even-better-toml
It's pyproject.toml-aware.

My linter showed the entire pyproject.toml file in red, so I went on the investigate.

Turns out both section and key names were incorrect. I fixed that. I also
removed the zip-safe flag because it's been deprecated for many years
and isn't used anymore.

BTW the linter I'm using is
https://marketplace.visualstudio.com/items?itemName=tamasfe.even-better-toml
@cbrnr cbrnr merged commit ac0d42c into mne-tools:main Nov 25, 2023
@cbrnr
Copy link
Contributor

cbrnr commented Nov 25, 2023

Thanks @hoechenberger!

@hoechenberger hoechenberger deleted the fix-pyproject branch November 25, 2023 14:35
larsoner added a commit to hoechenberger/mne-python that referenced this pull request Nov 27, 2023
* upstream/main:
  Add some omitted dependencies to "full" and "doc" variants (mne-tools#12235)
  Fix pyproject.toml setuptools configuration (mne-tools#12240)
  FIX: Allow eyetrack channels to be used with plot_compare_evoked (mne-tools#12190)
snwnde pushed a commit to snwnde/mne-python that referenced this pull request Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants