-
Notifications
You must be signed in to change notification settings - Fork 133
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
adding workflow for major release doc updates #4000
Conversation
Signed-off-by: anaxceron <[email protected]>
😺 Thank you for creating this PR! To publish your content to Zowe Docs, follow these steps.
|
1 similar comment
😺 Thank you for creating this PR! To publish your content to Zowe Docs, follow these steps.
|
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.
The overall flow makes sense to me 🙏
left a few questions/comments just to try and understand a bit better the whole process 😋
- Archive the `/static` files: | ||
1. Go to the `/static` folder. | ||
2. Create an empty directory with the name of the previous version in `/static`. For example: `static/v3.5.x`. | ||
3. Copy all contents of the `/static/stable` directory and paste them in the previous version's empty directory in the previous step. For example: `/static/v3.5.x`. |
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 it be possible to have these steps be done as part of the npm run <script>
mentioned above?
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.
Idk, but if so, we'd need dev help to look into, and implement, that.
2. Create an empty directory with the name of the previous version in `/static`. For example: `static/v3.5.x`. | ||
3. Copy all contents of the `/static/stable` directory and paste them in the previous version's empty directory in the previous step. For example: `/static/v3.5.x`. | ||
|
||
- Update a set of links in the archived documentation to ensure that they refer to the correct location: |
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.
Is there a way to run the link checker locally?
That way we don't have to rely on manual link-verification 😋
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.
It can be run locally, but would require installing extra deps like Rust/cargo.
cargo install lychee
lychee --accept 403,406,429 --cache --fallback-extensions md --include-fragments --max-cache-age 1d --max-concurrency 2 --timeout 60 --verbose 'docs/**/*.md'
|
||
1. To do this, switch to the archived directory. In our example, that's `versioned_docs/version-v3.5.x`. | ||
|
||
2. Update several locations in the archived docs to refer to the correct release, the 3.5 release (in our example case). |
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.
it sounds like these steps could also be moved into a script? 😋
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.
LGTM, thanks @anaxceron - I don't have any comments to add to Fernando's 😋
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.
Thanks Ana, LGTM
Signed-off-by: anaxceron <[email protected]>
Describe your pull request here: Documenting the process for updating doc for a major Zowe release
List the file(s) included in this PR: major_release_workflow.md
After creating the PR, follow the instructions in the comments.