Create documentation for 8.0.x and 9.x branch setup issue#44
Open
lachlan-robinson wants to merge 1 commit intothoth-tech:mainfrom
Open
Create documentation for 8.0.x and 9.x branch setup issue#44lachlan-robinson wants to merge 1 commit intothoth-tech:mainfrom
lachlan-robinson wants to merge 1 commit intothoth-tech:mainfrom
Conversation
✅ Deploy Preview for ontrackdocumentation ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
I'll be reviewing this PR as a senior. All in all, the documentation accurately details the steps required to properly enable branch switching to configure the development environment for Windows based machines. I can offer the following feedback, albeit being a little nit picky:
I'll approve this PR and let you consider the feedback above. Well done, keep it up. |
returnMarcco
approved these changes
Aug 4, 2025
ibi420
approved these changes
Sep 8, 2025
Contributor
ibi420
left a comment
There was a problem hiding this comment.
Hello Lachlan, your documentation clearly covers the required steps for building on both 8.x and 9.x, with appropriate visual aids. You’ve also modified the repository correctly. Thank you for the opportunity to review your work.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR adds a Windows-specific setup guide for the 8.x and 9.x branches. It documents the steps required to resolve environment issues when switching branches, including closing the dev container, deleting
package-lock.json, removing old Docker containers and volumes, updating dependencies and scripts inpackage.json, and restarting the development container. The guide ensures a smooth setup process for Windows users working with these branches.Type of change
How Has This Been Tested?
Testing Checklist
Checklist