Skip to content

Updates Docusaurus to 3.9.2#502

Merged
tfmorris merged 5 commits intoOpenRefine:masterfrom
thadguidry:update-docusaurus-1
Jan 28, 2026
Merged

Updates Docusaurus to 3.9.2#502
tfmorris merged 5 commits intoOpenRefine:masterfrom
thadguidry:update-docusaurus-1

Conversation

@thadguidry
Copy link
Copy Markdown
Member

- and prepares for Docusaurus 4.0.0
- and removes yarn (too many issues with Berry and PnP, and known issues with Docusaurus with yarn if not using node_modules facebook/docusaurus#10645 )
@netlify
Copy link
Copy Markdown

netlify Bot commented Jan 2, 2026

Deploy Preview for openrefine-website ready!

Name Link
🔨 Latest commit a8e15aa
🔍 Latest deploy log https://app.netlify.com/projects/openrefine-website/deploys/69795a92c11586000879500b
😎 Deploy Preview https://deploy-preview-502--openrefine-website.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 project configuration.

@thadguidry thadguidry requested a review from magdmartin January 2, 2026 03:21
Copy link
Copy Markdown
Member

@tfmorris tfmorris left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution! I've got a few questions and comments.

Comment thread README.md Outdated
Comment thread docusaurus.config.js
blogSidebarCount: 'ALL',
onInlineAuthors: 'ignore',

onUntruncatedBlogPosts: 'ignore'
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.

These are only warnings and seem like a useful nudge to the blog post writers to remember to include the truncation markers. What's the rationale for disabling the warnings?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I think @magdmartin didn't like the trunaction markers. So, ??? Not sure what to do here, up to both of you.

Comment thread docusaurus.config.js Outdated
Comment thread package.json Outdated
Comment on lines +35 to +36
},
"packageManager": "yarn@4.12.0"
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.

Suggested change
},
"packageManager": "yarn@4.12.0"
}

The PR comment says "removes Yarn" but this looks like it is actually adding Yarn. Let's keep that separate from the version updates. It's also looks like it could be the reason that CI/deploy is failing.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

ah, let me fix that!

thadguidry and others added 3 commits January 17, 2026 12:55
Co-authored-by: Tom Morris <tfmorris@gmail.com>
Co-authored-by: Tom Morris <tfmorris@gmail.com>
@tfmorris tfmorris self-requested a review January 28, 2026 00:33
Copy link
Copy Markdown
Member

@tfmorris tfmorris left a comment

Choose a reason for hiding this comment

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

Thanks!

@tfmorris tfmorris merged commit 23aa3fa into OpenRefine:master Jan 28, 2026
4 checks passed
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.

2 participants