Skip to content

Conversation

@hansvancalster
Copy link
Collaborator

@hansvancalster hansvancalster commented Jan 23, 2026

Description

Dit is een herneming van de publicatie van sop-040-nl #145 die eind vorig jaar faalde tijdens het bouwen van de website.

Related Issue

zie #145 voor de review historiek. Er is geen nieuwe inhoudelijke review nodig.

Task list

Steps by contributor:

  • Add description to this pull request (under "## Description")
  • Open the dropdown (triangle) near 'create pull request' and choose 'Create draft pull request'
  • Check for potential problems by running protocolhelper::check_frontmatter() and address them
  • Check for potential problems by running protocolhelper::check_structure() and address them
  • Add further commits if needed and push them to GitHub
  • Update the protocol-specific NEWS.Rmd
  • Mark the pull request as 'ready for review'

Review steps for the author(s):

  • Add reviewers, at least one subject-matter specialist and one administrator
  • Wait for review comments and address them
  • Iterate until reviewer approvals (merging the pull request will be done by an administrator)
  • Verify that the checks done by continuous integration succeeded. These will check if protocolhelper::check_frontmatter() and protocolhelper::check_structure() succeeded without errors.

To be done by an administrator after review: see guidelines for admins.

@hansvancalster hansvancalster requested a review from Anouk4 January 23, 2026 10:45
@hansvancalster
Copy link
Collaborator Author

@Anouk4 ik heb je als reviewer gevraagd (voor je eigen protocol 😄 ), maar dit is enkel omdat ik een pro-forma "approve" nodig heb. Hoe doe je dat: op deze pagina ga je naar "Files changed" tabblad en daar zal je zien dat je een "approve" kan geven. Ik heb goede hoop dat het deze keer wel zal lukken om je protocol op de website te krijgen 🤞

@hansvancalster hansvancalster merged commit 3210570 into main Jan 23, 2026
3 checks passed
@hansvancalster hansvancalster deleted the sop-040-nl branch January 23, 2026 12:48
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.

5 participants