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

Rewrite full documentation to VitePress #1121

Closed
Shinigami92 opened this issue Apr 26, 2024 · 2 comments · Fixed by #1132
Closed

Rewrite full documentation to VitePress #1121

Shinigami92 opened this issue Apr 26, 2024 · 2 comments · Fixed by #1132
Labels
c: docs Improvements or additions to documentation p: 1-normal Nothing urgent
Milestone

Comments

@Shinigami92
Copy link
Collaborator

Rewrite the full documentation to VitePress.

Needs to be done after a major bump to v8, cause we need to use "type": "module" / esm

@Shinigami92 Shinigami92 added the c: docs Improvements or additions to documentation label Apr 26, 2024
@Shinigami92 Shinigami92 added this to the vFuture milestone Apr 26, 2024
@Shinigami92 Shinigami92 added the p: 1-normal Nothing urgent label Apr 26, 2024
@Shinigami92 Shinigami92 modified the milestones: vFuture, v8.0 Apr 26, 2024
@brenoepics
Copy link
Collaborator

Hi @Shinigami92, I can do this, for the type module, we can just use config.mts and keep the type as common js or is v8 going to use type module for the pg migrate itself?

@Shinigami92
Copy link
Collaborator Author

v8 will switch to "type": "module" for the whole node-pg-mirgate codebase!

But anyway, if you would like to start converting the docs from docsify to vitepress, this would be an awesome help!

Maybe we can somehow server a compatibility until we switch fully to v8 as you suggested with a config.mts.

@Shinigami92 Shinigami92 linked a pull request Apr 27, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: docs Improvements or additions to documentation p: 1-normal Nothing urgent
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants