-
-
Notifications
You must be signed in to change notification settings - Fork 520
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
Vitepress build #2490
base: main
Are you sure you want to change the base?
Vitepress build #2490
Conversation
17df0da
to
b497fe3
Compare
Quality Gate passedIssues Measures |
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.
A few small comments. I think this will be good, thanks for that!
|
||
jobs: | ||
build: | ||
runs-on: ubuntu-latest |
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.
runs-on: ubuntu-latest | |
runs-on: ubuntu-24.04 |
I like to pin them, so they don't change from under you.
- name: Install dependencies | ||
run: yarn install --frozen-lockfile |
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.
Can we move the package.json, package-lock.json and yarn.lock files into docs? I'd like not to "pollute" the root if possible.
|
||
|
||
# vitepress - Docs guide | ||
node_modules/ |
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.
Would be nice if we can move the node_modules
into docs.
{ | ||
text: "v2", | ||
link: "https://mavsdk.mavlink.io/v2.0/en/cpp/api_changes.html", | ||
}, |
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.
I will create v3 and we should add it here, because main won't be v3 forever.
This is vitepress build from mavlink/MAVSDK-docs#292 adjusted to deploy from MAVSDK repo. It adds the docs to build into
/docs/
tree.Still to do
Not doing
This can be tested locally by cloning, then from root doing:
Then view in browser.