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

Test-info en installatiemethode #763

Merged
merged 9 commits into from
Jan 29, 2025
Merged

Conversation

marjoleintamis
Copy link
Contributor

@marjoleintamis marjoleintamis commented Dec 18, 2024

Info abacus-test.nl en installatiemethode toegevoegd

What's the scope of the changes?

  • Info over besturingssystemen aangepast (boven methode 1)
  • Methode 1 toegevoegd, nummering andere methoden gewijzigd
  • Methode 2 bijgewerkt
  • Info over Help toegevoegd
  • Plaatjes gewijzigd

What did you test? Which edge cases did you explicitly take into account?

n.v.t.

Are there things you want reviewers to definitely take a look at?

  • Even checken of de installatiemethoden ook op jouw computer zo werken, of dat er misschien afwijkingen zijn.
  • Eventueel gekke bewoordingen of onlogische volgorde van de tekst.

Are there specific people you want feedback from?

n.v.t.

Do you have tips on how to test? (For example: data setup, triggering specific errors, etc.)

Je kunt de instructies volgen en kijken of je ergens vastloopt als je ze op de letter volgt.

Draft, testinfo en installatiemethode toegevoegd
@marjoleintamis marjoleintamis added the documentation Improvements or additions to documentation label Dec 19, 2024
@marjoleintamis marjoleintamis linked an issue Dec 19, 2024 that may be closed by this pull request
- Info over besturingssystemen aangepast (boven methode 1)
- Methode 1 aangepast
- Methode 2 bijgewerkt
- Info over Help toegevoegd
- Plaatjes gewijzigd
Copy link

github-actions bot commented Jan 22, 2025

Sigrid maintainability feedback

💭️ You did not change any files that are measured by Sigrid


View this system in Sigrid

3 suggested changes

Co-authored-by: ring-ring-ring <76487016+ring-ring-ring@users.noreply.github.com>
Co-authored-by: Joep Schuurkes <145749778+jschuurk-kr@users.noreply.github.com>
- Uitproberen: 'wekelijkse release-builds' toegevoegd
- Methode 3: Abacus ipv productiebuild
- Instructies branch:main gewijzigd
- Plaatje aangepast
- Opmerking 'starten zonder argumenten' verplaatst
Volledige uitleg bij argumenten nu ook toegevoegd aan Windows
@marjoleintamis marjoleintamis added this pull request to the merge queue Jan 29, 2025
Merged via the queue into main with commit 8e5102c Jan 29, 2025
16 checks passed
@marjoleintamis marjoleintamis deleted the expand-install-methods branch January 29, 2025 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Doc 'Installeren en starten' uitbreiden
6 participants