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

Move vitepress to devDependencies, bump it to official v1.x.x release #105

Merged
merged 3 commits into from
May 7, 2024

Conversation

pascalwengerter
Copy link
Contributor

Fixes #104, also bumps Vitepress since it was still using a rc version.

I'm not a Vitepress expert by any means, but the only difference I could spot was a slightly larger top margin on the pages? Vitepress changelog notes two (unrelated) breaking changes since the formerly used release

@pascalwengerter
Copy link
Contributor Author

(Side note: Also not sure if docs/.vitepress/ or docs/.vitepress/cache necessarily need to be checked in with the rest of the repo?)

@MartinMalinda
Copy link
Owner

MartinMalinda commented May 3, 2024

Thanks a lot! I'm checking the deploy preview and it looks all right 👍

(Side note: Also not sure if docs/.vitepress/ or docs/.vitepress/cache necessarily need to be checked in with the rest of the repo?)

Ahh, judging by the number of lines changed (+14,278 −2,023), I guess no. Maybe I haven't fully followed some install instructions, it looks like the cache is something that's supposed to go to the gitignore.

Probably should be exactly the same as here: https://github.com/vitejs/vite/blob/main/.gitignore#L9

@pascalwengerter
Copy link
Contributor Author

Let me update the PR then ;)

@MartinMalinda
Copy link
Owner

Looks great, thanks a lot 🙏

@MartinMalinda MartinMalinda merged commit eaff780 into MartinMalinda:master May 7, 2024
5 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.

Why is Vitepress a normal dep?
2 participants