Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MAINT: Add and check type hints #995

Merged
merged 11 commits into from
Oct 26, 2024
Merged

MAINT: Add and check type hints #995

merged 11 commits into from
Oct 26, 2024

Conversation

larsoner
Copy link
Member

@larsoner larsoner commented Sep 11, 2024

Initial push to see if I broke anything.

  • Changelog has been updated (docs/source/changes.md)
  • I still had a function with no type annotations so need to tell mypy to be picky about those, too.

Closes #994

@larsoner larsoner marked this pull request as ready for review September 12, 2024 19:29
@larsoner
Copy link
Member Author

Green @hoechenberger !

@larsoner
Copy link
Member Author

@hoechenberger let me know if you want to look otherwise I'll go ahead and merge!

@hoechenberger
Copy link
Member

Hey, I won't have time to review until Friday or Saturday

will that suffice?

@larsoner
Copy link
Member Author

larsoner commented Oct 1, 2024

Sure!

@larsoner
Copy link
Member Author

@hoechenberger okay for me to fix the merge conflict and merge?

* upstream/main:
  [pre-commit.ci] pre-commit autoupdate (mne-tools#1006)
  [pre-commit.ci] pre-commit autoupdate (mne-tools#1002)
  allow missing sessions (mne-tools#1000)
  [pre-commit.ci] pre-commit autoupdate (mne-tools#1001)
@hoechenberger
Copy link
Member

Excellent work, @larsoner

@hoechenberger hoechenberger enabled auto-merge (squash) October 26, 2024 12:55
@hoechenberger hoechenberger merged commit c6bb948 into mne-tools:main Oct 26, 2024
51 of 52 checks passed
@larsoner larsoner deleted the mypy branch October 29, 2024 16:45
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.

ENH: Type checking
3 participants