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

Multilang #39

Merged
merged 4 commits into from
Oct 28, 2023
Merged

Multilang #39

merged 4 commits into from
Oct 28, 2023

Conversation

Thyrst
Copy link
Contributor

@Thyrst Thyrst commented Oct 24, 2023

#40

@netlify
Copy link

netlify bot commented Oct 24, 2023

Deploy Preview for sentence-splitter ready!

Name Link
🔨 Latest commit d6f8f43
🔍 Latest deploy log https://app.netlify.com/sites/sentence-splitter/deploys/653c63d959389f00081e048d
😎 Deploy Preview https://deploy-preview-39--sentence-splitter.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.

@azu azu marked this pull request as ready for review October 26, 2023 19:59
@azu azu added the Type: Feature New Feature label Oct 26, 2023
package.json Outdated
Comment on lines 22 to 35
"exports": {
".": {
"import": {
"types": "./module/sentence-splitter.d.ts",
"default": "./module/sentence-splitter.js"
},
"require": {
"types": "./lib/sentence-splitter.d.ts",
"default": "./lib/sentence-splitter.js"
},
"default": "./lib/sentence-splitter.js"
},
"./package.json": "./package.json"
},
Copy link
Member

Choose a reason for hiding this comment

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

What problems have you encountered with exports?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've managed to make it work. Just in my application's tsconfig.json I had to change moduleResolution from node to bundler.

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member

@azu azu left a comment

Choose a reason for hiding this comment

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

I believe this PR is a change to allow AbbrMarkerOptions to be passed from the outside.
This change itself is good because it was planned.
I had one question and I commented on it.

package.json Outdated Show resolved Hide resolved
@azu azu merged commit 40b8020 into textlint-rule:master Oct 28, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature New Feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants