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

Functiedocs toevoegen met XML #19

Open
Thom2503 opened this issue Apr 26, 2023 · 8 comments
Open

Functiedocs toevoegen met XML #19

Thom2503 opened this issue Apr 26, 2023 · 8 comments
Assignees
Labels
alle onderdelen documentation Improvements or additions to documentation

Comments

@Thom2503
Copy link
Collaborator

Thom2503 commented Apr 26, 2023

Dit is voor iedereen, misschien iets om op de achtergrond ook te doen, maar het is handig om overal waar methods staan functie docs te schrijven. Dus gewoon uitleggen in 1 zin wat de functie doet, met misschien een voorbeeld er bij.

/// <summary>
/// functie om 2 generic variables bij elkaar op te tellen.
/// </summary>
public T Add<T>(T first, T second)
{
    return first + second;
}
@Thom2503
Copy link
Collaborator Author

Weet trouwens niet of je automatisch closen van issues kan uitzetten wat hier niet zou moeten want dit is meer naslagwerk of iets wat je ff snel tussendoor commit.

@Thom2503 Thom2503 added the documentation Improvements or additions to documentation label Apr 26, 2023
@Peredurz
Copy link
Owner

Volgensmij gebeurt dat closen alleen als je iets doet met een commit en dan zegt #(number) is done etc.

@Thom2503
Copy link
Collaborator Author

Ja dat had Jurn uitgelegd ja maar het is misschien wel handig om het op een manier zeg maar aan te geven dat je naslagwerk doet voor deze issue.

@Thom2503
Copy link
Collaborator Author

Misschien als je iets commit wat met deze issue of een andere issue die niet gelijk moet closen iets van prefixen met [issue:19] en dan je commit message.

@Peredurz
Copy link
Owner

https://www.doxygen.nl/download.html, Dit is de website om Doxygen te installeren om goede documentatie te schrijven. Als je dit zelf wil installeren moet je naar beneden scrollen en bij de Sources en Binaries klikken op jouw operating system.

@Peredurz
Copy link
Owner

De docs kan je nu ook bekijken op deze website: https://peredurz.github.io/ProjectB/ .
Deze website hebben Thom en ik uren aan gezeten om het werkende te krijgen. Met bloed zweet en tranen

@Thom2503
Copy link
Collaborator Author

Oh ja, dus nu kan je voor de gedeeltes die /jij/ hebt gemaakt functie docs toevoegen, kijk ff bij PresentationLogic.cs hoe dat moet. En commit dat dan in de documentation branch, geef daarbij in de commit aan dat het voor deze issue is door [issue:19] te prefixen voor je commit message.

@Peredurz
Copy link
Owner

Je moet zelf de docs html lokaal bouwen met DoxyWizard met de Doxyfile die in de Documentation branch staat. Je moet zelf dan nog aangeven wat de working Directory van doxywizard is. Maar dan wordt er automatisch html code voor de github pages site gegenereerd. Als je dit vervolgens pusht wordt er door github actions automatisch de website gepubliceerd

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alle onderdelen documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

5 participants