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

Add draft of Full Change Log for 0.8.0 blog post #7

Merged
merged 5 commits into from
Jul 30, 2022
Merged

Add draft of Full Change Log for 0.8.0 blog post #7

merged 5 commits into from
Jul 30, 2022

Conversation

colepoirier
Copy link

Hey @cart, I tried my best to organize the sections by priority and add markdown strikethrough formatting and put at the bottom of each section the things that I thought should not be included. Obviously you'll have differing opinions on some of the determinations and changes I made; I hope this is still helpful. I didn't format the links yet because that will be easier to do once all of the inclusion determinations have been made by you.

Copy link

@alice-i-cecile alice-i-cecile left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Benchmarks, examples and tests are all internal and should be removed.

@colepoirier
Copy link
Author

Benchmarks, examples and tests are all internal and should be removed.

I think I got them all.

~~- Remove some unused dependencies (#4544)~~
~~- Misc dependency improvements (#4545)~~
~~- Decouple some dependencies (#3886)~~
~~- Update ndk-glue requirement from 0.5 to 0.6 (#3624)~~

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IMO dependency bumps should be listed, but likely in a separate section (or all together) to reduce noise.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good to me, but I'll leave that for @cart.

Copy link
Owner

@cart cart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was extremely helpful. Much appreciated @colepoirier!

@cart cart merged commit 22397a4 into cart:release-0.8.0 Jul 30, 2022
@colepoirier
Copy link
Author

Awesome! Glad it was helpful :)

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.

3 participants