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

Release 1.2.1 #324

Merged
merged 2 commits into from
Dec 22, 2022
Merged

Release 1.2.1 #324

merged 2 commits into from
Dec 22, 2022

Conversation

wuda-io
Copy link
Member

@wuda-io wuda-io commented Dec 14, 2022

Proposed changes

Releasing 1.2.1 Mainly because fixing the husky issue during installation #313

Screenshots (if appropriate) or codepen:

N/A

Types of changes

  • Bug fix (non-breaking change which fixes an issue).
  • New feature (non-breaking change which adds functionality).
  • Breaking change (fix or feature that would cause existing functionality to change).

Checklist:

  • I have read the CONTRIBUTING document.
  • My commit messages follows the conventional commit format
  • My change requires a change to the documentation, and updated it accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@wuda-io wuda-io changed the title chore: release 1.2.1 Release 1.2.1 Dec 14, 2022
@DanielRuf DanielRuf requested a review from a team December 15, 2022 16:21
@DanielRuf
Copy link

I have assigned @materializecss/members-write-access as reviewer since no one was informed due to the missing assignment.

@@ -49,14 +73,12 @@
- fix: Waves concat order [\#208](https://github.com/materializecss/materialize/pull/208) ([Smankusors](https://github.com/Smankusors))
- fix: for iOS13 selecting wrong Dropdownitem on Touchdevices fix \#31 [\#199](https://github.com/materializecss/materialize/pull/199) ([wuda-io](https://github.com/wuda-io))
- fix\(issue180\): handling invalid clickedIndex in carousel.js [\#187](https://github.com/materializecss/materialize/pull/187) ([LoganTann](https://github.com/LoganTann))
- Fix for issue \#146 \(Rounding artefacts with dark table row\) [\#149](https://github.com/materializecss/materialize/pull/149) ([RamonvdW](https://github.com/RamonvdW))

Choose a reason for hiding this comment

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

Hm, how can it happen that now we are missing some old entries?

Copy link
Member

Choose a reason for hiding this comment

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

oh, I forgot to mention this... the CHANGELOG needs a little manual fixes 😅

this one is missing because it could not find the commit on the main branch

I'm not sure how this happen though, but this PR changes has been bought by 55e4854 now

@Smankusors
Copy link
Member

anyway, to reduce the manual work for CHANGELOG, how about if we include only from 1.2.0 to 1.2.1?

for the earlier versions, the user can read on releases page

wdyt?

@wuda-io
Copy link
Member Author

wuda-io commented Dec 16, 2022

anyway, to reduce the manual work for CHANGELOG, how about if we include only from 1.2.0 to 1.2.1?

for the earlier versions, the user can read on releases page

wdyt?

Yes thats better 👍, it should list only the differences/changes between 2 releases. Generating the whole log each time and overwrite it every release is not very efficient.

@wuda-io
Copy link
Member Author

wuda-io commented Dec 16, 2022

I have assigned @materializecss/members-write-access as reviewer since no one was informed due to the missing assignment.

Oh thats correct I forgot that, thanks!

@DanielRuf
Copy link

Oh thats correct I forgot that, thanks!

Regarding assignments, I will check if we can enable some sort of auto-assignments. So no PR will be missed then in the future.

@wuda-io
Copy link
Member Author

wuda-io commented Dec 19, 2022

Ok, please review, I have updated the changelog with only the differences since the last version (1.2.0)

@DanielRuf
Copy link

Ok, please review, I have updated the changelog with only the differences since the last version (1.2.0)

Seems now all the old changelog entries are gone.

@wuda-io
Copy link
Member Author

wuda-io commented Dec 19, 2022

Seems now all the old changelog entries are gone.

Yes, there was an error when generating the whole changelog, some PRs were not found and needed manual fixing.
@Smankusors suggested to only list the changes between two releases. (from 1.2.0 to 1.2.1) I also think this is the better approach. When looking up older changes this can be done via the release-page. Wdyt?

@Smankusors
Copy link
Member

Smankusors commented Dec 19, 2022

ooh! I just realized that we can use --base CHANGELOG.md to append the changelog, then there is no manual work anymore 😳

hmm but I'm still not sure if we should maintain a full changelog? I'm curious since we have a releases page, what's the purpose of the CHANGELOG.md anymore other than for drafting the release? 😅

@DanielRuf
Copy link

Changelog via GH interface and releases sounds way easier and better. Afaik GH supports the creation of the release changelog via click.

If this is the best option for us, then we can empty changelog.md and link there to https://github.com/materializecss/materialize/releases/tag/1.2.1 in there.

Copy link
Member

@Smankusors Smankusors left a comment

Choose a reason for hiding this comment

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

LGTM

@wuda-io
Copy link
Member Author

wuda-io commented Dec 22, 2022

Changelog via GH interface and releases sounds way easier and better. Afaik GH supports the creation of the release changelog via click.

If this is the best option for us, then we can empty changelog.md and link there to https://github.com/materializecss/materialize/releases/tag/1.2.1 in there.

I will merge this now. We also can further improve the changelog handling, let us discuss ideas and improvements in a speperate issue. Thanks and happy holidays to all of you!

@wuda-io wuda-io merged commit e4e3696 into materializecss:main Dec 22, 2022
@wuda-io wuda-io deleted the release-1.2.1 branch December 22, 2022 07:15
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