Open
Conversation
✅ Deploy Preview for thoth-tech ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
WaelAlahamdi
approved these changes
Mar 30, 2026
Contributor
WaelAlahamdi
left a comment
WaelAlahamdi
left a comment
There was a problem hiding this comment.
The review was done locally. I checked the pull request branch, successfully ran npm install and npm run build, and reviewed the code changes. I found no issues or comments.
I also visually checked several pages to ensure the formatting looks correct, including:
- /index.html
- /products/ontrack/01-start-contributing/
- /products/ontrack/02-set-up-dev/
- /products/ontrack/06-pull-request-template/
- /resources/privacy-policy/
I also reviewed the changes across all modified files and did not find any missing text in the edits. Everything appears to be working correctly.
Collaborator
Author
|
@BrianDangDev please approve if appropriate. |
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
2 participants
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 change is intended to make the source code of this repo simpler to read in a terminal. Primarily, the motivation is to change any line greater than 80 characters back to 80 characters at most. There are many justifications, first of which is that Prettier's default printWidth is 80 characters. When using
cat <file>the entire Markdown document should be able to be read in the system terminal.It's not super worth looking through the diff for this change, there have been many files changed.
Type of change
How Has This Been Tested?
No issues when
npm run buildis run. No formatting issues created as a resultTesting Checklist