fixed string subID bug

[Aaronearlerichardson]

PR Description

• fixes Failed to create new participants.tsv entry when subject ID is non numeric #1290

Merge checklist

Maintainer, please confirm the following before merging.
If applicable:

• All comments are resolved
• This is not your own PR
• All CIs are happy
• PR title starts with [MRG]
• whats_new.rst is updated
• New contributors have been added to CITATION.cff
• PR description includes phrase "closes <#issue-number>"

[welcome]

Hello! 👋 Thanks for opening your first pull request here!
Please read the contributor guide, and please follow the steps outlined in the "Instructions for first-time contributors" section therein. ❤️ We will try to get back to you soon. 🚴🏽‍♂️

[Aaronearlerichardson]

will fix #1290

[sappelhoff]

Thanks @Aaronearlerichardson! Could you please modify one of our tests to cover this case?

Also, please feel free to add yourself to the list of authors, and add a changelog entry: https://github.com/mne-tools/mne-bids/blob/main/CONTRIBUTING.md#instructions-for-first-time-contributors

[sappelhoff]

@Aaronearlerichardson please let me know if you intend to work on the final points I mentioned in my comment above, and what your expected timeline would be. If not, somebody else could perhaps take over (as long as you supply a few of your details to list you in the authors lists)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.46%. Comparing base (87eea28) to head (1817b34).
Report is 60 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1291      +/-   ##
==========================================
- Coverage   97.61%   97.46%   -0.16%     
==========================================
  Files          40       40              
  Lines        8685     8780      +95     
==========================================
+ Hits         8478     8557      +79     
- Misses        207      223      +16     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Aaronearlerichardson]

@sappelhoff hope this covers it

[welcome]

🎉 Congrats on merging your first pull request! 🥳 Looking forward to seeing more from you in the future! 💪

[sappelhoff]

Thanks a lot @Aaronearlerichardson :-)

@sappelhoff hope this covers it

I added a few more commits, but thanks for the start and the contribution in general