-
-
Notifications
You must be signed in to change notification settings - Fork 223
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
Next release planning #451
Comments
Here's a few from my TODO.md:
Then of course also doing the ones from #427, that we agreed on. Though overall I just wanna iron out the bugs. Been a bit busy for the past month, but I think I'll be able to fix those bugs in the upcoming weekend. |
Sounds good -- let's see where we are on Monday. |
One of them was already fixed, but I've fixed the 3 other bugs. I also discovered an additional bug (I've edited the previous comment). |
I think it's past time we got this out. @vallentin when would you have time to investigate #494? |
If it can wait until Thursday / the weekend, then yes :) |
Yeah, that should be fine. |
I'll see if I can spend some time on this tomorrow (mostly writing up a change log). |
Was #494 the "last" important bug, before the new release? Just glancing through this issue and #427, then it seems the remaining stuff is new features. When you start writing the change log, feel free to make a draft PR, then I'll comment any missing stuff I recall. I guess one of the most important things, is highlighting the change to the implicit borrowing. Mentioning that literal primitives and the result of functions are no longer borrowed. |
At this point I'll probably wait for the nom 7 release, which should help with the silly dependencies. |
Since nom 7 has been released, is there anything else blocking a release? |
I wrote up draft release notes: https://github.com/djc/askama/releases/edit/untagged-1f87fb7743ec53e520b9. @vallentin and @Kijewski, would you please review to see if anything important is missing or should be highlighted more? |
Maybe the "General changes" should be split up, so e.g. bug fixes are moved to the bottom of the description, e.g. #394 most probably haven't encountered this. Then "General changes" could be easier to glance through, so users can more easily spot the "real" changes, that affect them or they're interested about |
A project this big can never be free of errors, that will only show themselves after the release. I think the changelog addresses to the users' interests quite well. 👍 from me! |
@vallentin I forget what things I still wanted to resolve before releasing... Do you have anything else you would like to land in the short term?
The text was updated successfully, but these errors were encountered: