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

refactor!: Use a wildcard export for ProseMirror model, state and view #733

Merged
merged 1 commit into from
Apr 11, 2024

Conversation

rfgamaral
Copy link
Member

Overview

More often than not, we need to import things from ProseMirror related packages on our consumer applications, and so far we've been exporting exactly what we need when needed. This slows us down a bit, because it forces us to do some work on Typist when something we need is not yet exported.

We (I discussed this with Pedro) decided that it would be best to remove this step in the workflow, and for that reason we will now start exporting everything in the @tiptap/pm/* packages as wildcard ProseMirror* exports.

PR Checklist

@rfgamaral rfgamaral added the ⛴ Ship PR Used for PRs that don't need a review and can be merged when CI is green. label Apr 11, 2024
@rfgamaral rfgamaral self-assigned this Apr 11, 2024
@rfgamaral rfgamaral force-pushed the ricardo/refactor-prosemirror-exports branch from 76d05f0 to b752f65 Compare April 11, 2024 08:23
Copy link

netlify bot commented Apr 11, 2024

Deploy Preview for doist-typist ready!

Name Link
🔨 Latest commit 76d05f0
🔍 Latest deploy log https://app.netlify.com/sites/doist-typist/deploys/66179de640b06c000826be0c
😎 Deploy Preview https://deploy-preview-733--doist-typist.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Apr 11, 2024

Deploy Preview for doist-typist ready!

Name Link
🔨 Latest commit 52e915e
🔍 Latest deploy log https://app.netlify.com/sites/doist-typist/deploys/66179f1a8386dd0008b02d22
😎 Deploy Preview https://deploy-preview-733--doist-typist.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@rfgamaral rfgamaral force-pushed the ricardo/refactor-prosemirror-exports branch from b752f65 to 52e915e Compare April 11, 2024 08:28
@rfgamaral rfgamaral merged commit 496e976 into main Apr 11, 2024
9 checks passed
@rfgamaral rfgamaral deleted the ricardo/refactor-prosemirror-exports branch April 11, 2024 08:33
doistbot added a commit that referenced this pull request Apr 11, 2024
## [6.0.0](v5.0.1...v6.0.0) (2024-04-11)

### ⚠ BREAKING CHANGES

* Use a wildcard export for ProseMirror model, state and view (#733)

### Code Refactoring

* Use a wildcard export for ProseMirror model, state and view ([#733](#733)) ([496e976](496e976))
@doistbot
Copy link
Member

🎉 This PR is included in version 6.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@doistbot doistbot added the released Pull requests that have been released to production label Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released Pull requests that have been released to production ⛴ Ship PR Used for PRs that don't need a review and can be merged when CI is green.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants