Skip to content

Conversation

@BattleCh1cken
Copy link
Contributor

@BattleCh1cken BattleCh1cken commented May 8, 2024

Summary

This pull request migrates the documentation from a .pdf file, which is hard to navigate for users, to a website rendered by mdbook.

Checklist

  • I have fully tested all of the added features
  • I have fully documented all of the relevant code

@BattleCh1cken
Copy link
Contributor Author

This pr is blocked by #38 to avoid merge conflicts.

@BattleCh1cken BattleCh1cken force-pushed the feature/documentation-overhaul branch from 3cd4ad5 to f6b778b Compare May 9, 2024 21:16
@BattleCh1cken BattleCh1cken marked this pull request as ready for review May 14, 2024 20:44
@BattleCh1cken BattleCh1cken requested review from Isaiah15414, aji-ajani and dblaufuss and removed request for dblaufuss May 15, 2024 21:34
- `globals.typ`: Contains all of the global variables for the entire project.
- `utils.typ`: Utility functions intended to help implement themes.
- `themes/`: The folder containing all of the themes.
- `themes.typ`: An index of all the themes are contained in the template
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove tab

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it has a tab because themes.typ is inside the themes/ folder

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I realize my mdbook was out of date.

Copy link
Collaborator

@Isaiah15414 Isaiah15414 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! Just fixed some typos, but everything reads very well

@BattleCh1cken BattleCh1cken force-pushed the feature/documentation-overhaul branch from a1447a7 to 084b690 Compare May 15, 2024 22:49
@BattleCh1cken BattleCh1cken merged commit 90b2af5 into main May 15, 2024
@BattleCh1cken BattleCh1cken deleted the feature/documentation-overhaul branch May 16, 2024 14:53
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.

4 participants