Skip to content
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

[WIP] [Docs] Add conventions documentation #4718

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Nitemaeric
Copy link
Contributor

@Nitemaeric Nitemaeric commented Nov 27, 2024

Context

We want to align on how the Publish Teacher Training code should be written.

Update: In order to turn this into a collaboratively iterative process, let's start here and add more content in separate PRs.

Changes proposed in this pull request

  • Add some conventions documentation.

Guidance to review

View the documentation here:

@Nitemaeric Nitemaeric self-assigned this Nov 27, 2024
@Nitemaeric Nitemaeric changed the title [WIP] Add conventions documentation [WIP] [Docs] Add conventions documentation Nov 27, 2024
@Nitemaeric Nitemaeric force-pushed the dd/application-architecture-and-conventions branch from dd2d946 to 9717ef7 Compare November 27, 2024 16:07
@Nitemaeric Nitemaeric force-pushed the dd/application-architecture-and-conventions branch 5 times, most recently from 472f038 to 56b2d28 Compare December 18, 2024 14:11
@Nitemaeric Nitemaeric force-pushed the dd/application-architecture-and-conventions branch 2 times, most recently from 6acb1b7 to 36c209e Compare January 28, 2025 12:58
@Nitemaeric Nitemaeric force-pushed the dd/application-architecture-and-conventions branch from 36c209e to 24707cd Compare March 14, 2025 14:23
@Nitemaeric Nitemaeric force-pushed the dd/application-architecture-and-conventions branch from 24707cd to fc8c0be Compare March 27, 2025 15:08
@Nitemaeric Nitemaeric marked this pull request as ready for review March 27, 2025 15:14
@Nitemaeric Nitemaeric requested a review from a team as a code owner March 27, 2025 15:14
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.

1 participant