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

Move @emotion/styled-base to @emotion/styled/base #1575

Merged
merged 18 commits into from
Nov 8, 2019

Conversation

emmatown
Copy link
Member

@emmatown emmatown commented Oct 27, 2019

What:

Move @emotion/styled-base to @emotion/styled/base

Why:

So that package authors who are using @emotion/styled and babel-plugin-emotion don't need to also add @emotion/styled-base to their dependencies.(if they didn't, their packages would not work in more strict package managers/bundlers)

How:

Use a Preconstruct entrypoint

Checklist:

  • Documentation
  • Tests
  • Code complete
  • Changeset

@emmatown emmatown added this to the v11 milestone Oct 27, 2019
@changeset-bot
Copy link

changeset-bot bot commented Oct 27, 2019

🦋 Changeset is good to go

Latest commit: ae18a8b

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

@FezVrasta
Copy link
Member

That’s great! Thanks

"module": "dist/styled.esm.js",
"umd:main": "dist/styled.umd.min.js",
"browser": {
"./dist/styled.cjs.js": "./dist/styled.browser.cjs.js",
Copy link
Member

Choose a reason for hiding this comment

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

i'm not entirely sure about it right now - but I vaguely remember at least some tools ignoring this alias field for extra entries, and that I had to include those redirects in root package.json

# Conflicts:
#	packages/styled-base/CHANGELOG.md
#	packages/styled-base/package.json
#	packages/styled/package.json
#	yarn.lock
@codesandbox-ci
Copy link

codesandbox-ci bot commented Nov 6, 2019

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit ae18a8b:

Sandbox Source
Emotion Configuration

@Andarist Andarist changed the base branch from master to next November 6, 2019 18:33
@Andarist
Copy link
Member

Andarist commented Nov 6, 2019

@mitchellhamilton I think this is nearly done, just surprisingly some flow errors popped up. I've asked @FezVrasta for help with this.

FezVrasta and others added 6 commits November 7, 2019 14:00
# Conflicts:
#	package.json
#	packages/babel-plugin-emotion/__tests__/__snapshots__/styled.js.snap
#	packages/core/__tests__/css.js
#	packages/primitives/test/emotion-primitives.test.js
#	yarn.lock
@emmatown emmatown merged commit f9feab1 into next Nov 8, 2019
@emmatown emmatown deleted the styled-base-as-an-entrypoint branch November 8, 2019 21:42
louisgv pushed a commit to louisgv/emotion that referenced this pull request Sep 6, 2020
* Change @emotion/styled-base to @emotion/styled/base

* Fix some stuff

* Fix some stuff

* Fix another thing

* Remove styled-base from CodeSandbox CI config

* Make @emotion/styled/base TS types working

* Restore flowtype tests for @emotion/styled/base

* Fix last reference to @emotion/styled-base

* fix some flow errors

* Brought back @emotion/styled-base, but throw a deprecation error at runtime

* fix styled Flow tests (emotion-js#1615)

* fix flow type error

* Add changeset

* Update spotty-llamas-mate.md

* Update index.js

* Remove @emotion/styled-base from some changesets
@github-actions github-actions bot mentioned this pull request Nov 10, 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

3 participants