-
Notifications
You must be signed in to change notification settings - Fork 165
Switch mui-datatables to actively maintained leecalcote fork #1218
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
Conversation
Co-authored-by: yi-nuo426 <[email protected]>
Co-authored-by: yi-nuo426 <[email protected]>
|
@copilot, refresh, pulling the latest code from leecalcote/mui-datatables |
Co-authored-by: yi-nuo426 <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR switches from the unmaintained mui-datatables package to an actively maintained fork by Lee Calcote to address the lack of maintenance on the upstream repository (no activity for 3+ years).
Changes:
- Updated
mui-datatablesdependency from npm version^4.3.0to GitHub fork pinned at commite09c39cb - Applied changes to both root package.json and examples/next-12/package.json
- Updated package-lock.json to reference the GitHub repository
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| package.json | Updated both devDependencies and dependencies entries to use the GitHub fork URL |
| package-lock.json | Updated resolved URL to point to the GitHub fork repository |
| examples/next-12/package.json | Updated dependency to match the fork reference |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Yi Nuo <[email protected]>
Signed-off-by: Lee Calcote <[email protected]>
This package was causing npm dependency resolution conflicts after PR #1218 switched to the GitHub fork of mui-datatables. The code now uses the 'mui-datatables' package from github:leecalcote/mui-datatables, so the old @sistent/mui-datatables package in devDependencies is no longer needed. Co-authored-by: fitzergerald <[email protected]>
github:leecalcote/mui-datatables#decbb0493fdd95d674d2dd748d3d35cefb801195) instead of npm registry versiondecbb049from leecalcote/mui-datatables forkOriginal prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.