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.

Sign up for GitHub

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

Jump to bottom

Improve documentation coverage #316

Merged
merged 13 commits into from
Aug 23, 2021
Merged

Improve documentation coverage #316

merged 13 commits into from
Aug 23, 2021

Conversation

JonasMuehlmann
Copy link
Owner

@JonasMuehlmann JonasMuehlmann commented Aug 16, 2021 •
edited
Loading

Before this PR can be merged, the following checkboxes have to be ticked

  • This code belongs to the backend and is tested, or tests are not useful
  • New functions with more than 5 lines have docstrings
  • New code has been integrated with the MessengerService if appropiate
  • New code has been integrated with the SignalRService if appropiate
  • New functions or classes have been documented in the Entwurfsdokumentation
  • Changes introduce no commented out code or Console.Writeline() statements used for debugging

Related issues:

Closes #221

@JonasMuehlmann JonasMuehlmann linked an issue Aug 16, 2021 that may be closed by this pull request
Improve documentation coverage #221
Closed
@JonasMuehlmann JonasMuehlmann changed the title docs: add README.md files to explain directory structure Improve documentation coverage Aug 16, 2021
@JonasMuehlmann JonasMuehlmann mentioned this pull request Aug 16, 2021
Merged
@JonasMuehlmann JonasMuehlmann linked an issue Aug 17, 2021 that may be closed by this pull request
Add readme files to explain directory and file structure #153
Open
@JonasMuehlmann JonasMuehlmann merged commit d24dca3 into unstable Aug 23, 2021
@JonasMuehlmann JonasMuehlmann deleted the improve-doc-coverag branch September 24, 2021 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@robbelDev robbelDev robbelDev approved these changes

Assignees

@JonasMuehlmann JonasMuehlmann

@JayKim94 JayKim94

Labels
Backend implemented status: in development
Projects
None yet
Milestone
Final Feature and Documentation Sprint
Development

Successfully merging this pull request may close these issues.

Add readme files to explain directory and file structure Improve documentation coverage
3 participants
@JonasMuehlmann @robbelDev @JayKim94