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

Accept objects as className in jsx-created components (they are stringified) to match React behavior #1548

Merged
merged 2 commits into from
Oct 21, 2019

Conversation

Andarist
Copy link
Member

Followup to #1545

@changeset-bot
Copy link

changeset-bot bot commented Oct 11, 2019

🦋 Changeset is good to go

Latest commit: e5f7782

We got this.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@codecov
Copy link

codecov bot commented Oct 11, 2019

Codecov Report

Merging #1548 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted Files Coverage Δ
packages/styled-base/src/index.js 100% <100%> (ø) ⬆️
packages/core/src/jsx.js 100% <100%> (ø) ⬆️

Copy link
Member

@emmatown emmatown left a comment

Choose a reason for hiding this comment

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

Thanks!!

@Andarist
Copy link
Member Author

Once PRs get approved can we merge them right away? Or do you prefer to keep them open for some reason?

@emmatown emmatown merged commit 2fc75f2 into master Oct 21, 2019
@emmatown emmatown deleted the unify-jsx-n-styled-classname-handling branch October 21, 2019 10:06
@emmatown
Copy link
Member

You can merge them right away when they're approved, the usual reason why I don't is because I'm waiting on CI and get distracted

@Andarist
Copy link
Member Author

Thanks for the reply! Roger that 😉

@github-actions github-actions bot mentioned this pull request Oct 21, 2019
louisgv pushed a commit to louisgv/emotion that referenced this pull request Sep 6, 2020
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.

None yet

2 participants