Skip to content

chore: remove mintlify dev dep#1554

Open
nlucero wants to merge 1 commit into
mainfrom
chore/remove-mintlify-dev-dep
Open

chore: remove mintlify dev dep#1554
nlucero wants to merge 1 commit into
mainfrom
chore/remove-mintlify-dev-dep

Conversation

@nlucero

@nlucero nlucero commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Description

References

Testing

All universal-components npm scripts run successfully

Checklist

  • I've read and followed CONTRIBUTING.md.
  • I've tested the site build for this change locally.
  • I've made appropriate docs updates for any code or config changes.
  • I've coordinated with the Product Docs and/or Docs Management team about non-trivial changes.

@nlucero nlucero self-assigned this Jul 15, 2026
@nlucero
nlucero requested a review from a team as a code owner July 15, 2026 15:14
@nlucero nlucero added the dependencies Pull requests that update a dependency file label Jul 15, 2026

@grandmaester grandmaester left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me.
We can remove this unused dev dependency as mintlify is a global dependency.
But I guess there is a check added for UI Build which mandates if you are changing something (code, deps) on universal-components repo, it requires a new package update.
Please go ahead and run below steps:

  1. Increment minor version in package.json of universal-components repo
  2. Run pnpm run build to build the components assets which also copies it to main
  3. Commit the Bundled assets created along with package.json file so the build check here pass.

@grandmaester grandmaester left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Comment added above

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants