Skip to content

Create initial Dev Docs - #187

Draft
justinschwerdtfeger wants to merge 44 commits into
developfrom
dev-docs
Draft

justinschwerdtfeger wants to merge 44 commits into
developfrom
dev-docs

Conversation

@justinschwerdtfeger

@justinschwerdtfeger justinschwerdtfeger commented Sep 2, 2026 •

Copy link
Copy Markdown
Contributor

Description

Adds developer documentation to Github Wiki using Andrew-Chen-Wang/github-wiki-action@v5 GitHub action.

Testing

AI Disclosure

I used AI to write initial docs based on my layout, then heavily edited them.

Checklist

  • Create this PR
  • Make GitHub Action
  • Finish writing Home.md, Getting-Started.md, and How-to-Contribute.md
  • Finish writing Docker-Guide.md, Project-Structure.md, and Troubleshooting.md
  • Update README.md
  • Perform final self-review

@justinschwerdtfeger

Copy link
Copy Markdown
Contributor Author

@Yogeb-tech Can you finish this up?

@Yogeb-tech

Copy link
Copy Markdown
Contributor

Yes, ideally I will have it done by the end of this week

@justinschwerdtfeger

justinschwerdtfeger commented Sep 2, 2026 •

Copy link
Copy Markdown
Contributor Author

Windows build is failing #188. We definetly want to get a fix in

@justinschwerdtfeger justinschwerdtfeger changed the title Dev docs Create initial Dev Docs Sep 3, 2026
@justinschwerdtfeger

Copy link
Copy Markdown
Contributor Author

just a random thought,

the AI I was using to make initial drafts wrote a lot of numbers in the headers. I don't really like this, and I was working to remove most of them

@justinschwerdtfeger
justinschwerdtfeger removed this pull request from stack #190 September 10, 2026 20:14
Update documentation, especially project architecture and structure pages
Comment thread README.md

| Command | Description |
| ---------------------------------------------- | ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
| `./api-tools --scrape --coursebook --term 24F` | Scrapes CourseBook data for Fall 2024.<br>• `--resume`: Resume from last completed course prefix.<br>• `--startprefix <prefix>`: Begin at a specific course prefix (e.g. `cp_cs`). |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This example command won't work because the term is invalid. Must be a lowercase f, not uppercase F.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

good catch

Comment thread README.md
| Command | Description |
| ---------------------------------------------- | ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
| `./api-tools --scrape --coursebook --term 24F` | Scrapes CourseBook data for Fall 2024.<br>• `--resume`: Resume from last completed course prefix.<br>• `--startprefix <prefix>`: Begin at a specific course prefix (e.g. `cp_cs`). |
| `./api-tools --scrape --evals --term 24F` | Scrapes CourseBook course evaluation data for the specified term. |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Don't know what format term currently uses for the evaluation data, but we should try to keep it the same as with coursebook.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'm not sure what you mean @KBui4

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I meant that we shouldn't have:
./api-tools --scrape --coursebook --term 24f
./api-tools --scrape --evals --term 24F

Where one uses a lowercase f, s, or u, and the other uses uppercase. Both should follow the same format.

@AbhiramTadepalli AbhiramTadepalli left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

i like the structure

Comment thread docs/Home.md
Comment thread docs/How-to-Contribute.md

This branch has not been deployed

No deployments
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