Skip to content

Bump classnames from 2.2.5 to 2.3.2 - #51211

Merged
levadadenys merged 3 commits into
stagingfrom
denys/sl/update-classNames
Apr 10, 2023
Merged

Bump classnames from 2.2.5 to 2.3.2#51211
levadadenys merged 3 commits into
stagingfrom
denys/sl/update-classNames

Conversation

@levadadenys

Copy link
Copy Markdown
Contributor

Bump classnames from 2.2.5 to 2.3.2

Simple bump of classnames lib version.
No breaking changes intended, just a small improvement for Typescript dev experience. (In this version lib has types for TS, previously it didn't have them)
Now we can use import classNames from "classnames"; instead of const classNames = require('classnames');

Links

Testing story

Deployment strategy

Follow-up work

Privacy

Security

Caching

PR Checklist:

  • Tests provide adequate coverage
  • Privacy and Security impacts have been assessed
  • Code is well-commented
  • New features are translatable or updates will not break translations
  • Relevant documentation has been added or updated
  • User impact is well-understood and desirable
  • Pull Request is labeled appropriately
  • Follow-up work items (including potential tech debt) are tracked and linked

@levadadenys
levadadenys requested review from a team, breville and sanchitmalhotra126 April 7, 2023 17:35

@molly-moen molly-moen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🎉

Comment thread apps/src/music/views/ChordPanel.tsx Outdated

@breville breville left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Not very familiar with this component, but Music Lab changes look fine. Thanks!

@sanchitmalhotra126 sanchitmalhotra126 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for doing this!

@levadadenys
levadadenys merged commit 4faf13e into staging Apr 10, 2023
@levadadenys
levadadenys deleted the denys/sl/update-classNames branch April 10, 2023 15:07
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.

4 participants