-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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: Release 0.21 #8150
Comments
+10 !
|
September 15 ? @larsoner |
Whoops, yes I meant Sept 15th :) |
We need to draft an announcement. Also with the last release we had this small video clip demonstrating the new Time Viewer. We should do a similar demo of some notable features in 0.21 as well. Which are the most notable new features? |
Maybe:
|
Maybe |
From the chanelog:
|
|
BTW we are not ready to release, I'm still doing battle with CircleCI |
@hoechenberger since we're already discussing the release, did you have time to work on an |
@cbrnr Yes, in conda-forge/mne-feedstock#37 |
If possible I'd like us to start highlighting the contributions of new-to-mne developers in our release announcements. At a minimum, maybe "there were N contributions from M first-time contributors in this release cycle". Better would be to also give examples and call out their names. |
Maybe we should start this process by bolding the entries in the |
That could work, although I wonder if folks read the whole changelog (i.e., would bolding actually increase visibility if it's not being read?) |
Some people might, either way they get more visible and next time it's easier to know which entries we should add based on being from new contributors next time. To be clear, I'm suggesting that we just start by doing the bolding in |
Ah, got it. Seems like a good approach |
Should we also add the related PRs to all new entries? This would be very handy for people who would like to dig into the corresponding code changes. |
If someone wants to do this, feel free. I'm not up for it, I suspect it's going to take some time... the fastest way I can think of is to look at the Another option going forward in other repos I've used is to have maintainers ensure that the title of the PR is good and that it's tagged with |
I was thinking of adding the PR only for new changes going forward - I wouldn't want to do it for all changes (in 0.21) either. As long as we manually create changelog entries, it should not be too much effort for the contributor to also add the corresponding PR. If there is a way to automatically generate the complete changelog - yes, we should go for it, but this seems like it will probably need more time until it works. |
we are doing this "manual" changelog with PR links in MNE-BIDS, see changelog. I would be in favor of continuing the changelog manually and simply adding a PR link "on top". The effort is low, and the benefit is tangible. re: automatic changelog generation --> would it save a lot of time? would it be of an equally high quality? ... or will it turn out to be a lot of work for something that's "almost as good as manual"? |
This is an automatically generated one: https://sphinx-gallery.github.io/stable/changes.html So basically we would set the PR title to what we want the changelog entry to be, and instead of putting it in a section we tag it with a GitHub label. So the quality depends on how nicely you format your PR titles.
In general it does save work for contributors because there is no need for rebase/merge due to latest.inc conflicts. For maintainers it only adds a couple minutes during release. So I see it as a net time saver. One downside I see is that you really only get it on release rather than continuously. But perhaps worse, we would lose the |
I just wanted to share that I very much like this idea, particularly in the context of #8221 |
Its probably not as widely used as the features hoechenberger highlighted, but support for NIRS over 0.20 and 0.21 has increased. Slight bias here. |
We should cut a release in a few weeks, tentatively scheduled for September 15th. Feel free to tag issues and PRs that realistically can and should make it if you have permissions, and if you don't, feel free to comment on ones that you think should be tagged for 0.21.
The text was updated successfully, but these errors were encountered: