Skip to content

Sfp 101 nl - #159

Merged
hansvancalster merged 10 commits into
mainfrom
sfp-101-nl
Aug 3, 2026
Merged

Sfp 101 nl#159
hansvancalster merged 10 commits into
mainfrom
sfp-101-nl

Conversation

@Florian9041

@Florian9041 Florian9041 commented Apr 30, 2026

Copy link
Copy Markdown
Contributor

Description

Veldprotocol quickscan knijten. Protocol veldstaalname en laboverwerking.
INBOPRJ-23055
PRJ - Knijten - Culicoides onderzoek in het Schelde-estuarium (EVINBO) Team Estuaria.

Related Issue

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.

@Florian9041 Florian9041 self-assigned this Apr 30, 2026
@Florian9041
Florian9041 marked this pull request as ready for review June 3, 2026 07:33
@Florian9041

Copy link
Copy Markdown
Contributor Author

@merlijn-jocque kan je dit protocol doornemen als je tijd hebt?
Als ik je goedkeuring heb ("approve changes" of iets dergelijks), dan kan dit gepubliceerd worden.

@hansvancalster

Copy link
Copy Markdown
Collaborator

Vermoedelijk werd deze pull request per ongeluk gesloten. Ik zal hem terug heropenen.

@hansvancalster hansvancalster reopened this Aug 3, 2026
@Florian9041

Copy link
Copy Markdown
Contributor Author

@hansvancalster klopt, ik ging je hier nog een mail voor sturen. Merci om dit recht te zetten!

@Florian9041

Copy link
Copy Markdown
Contributor Author

Merlijn heeft dit ook gereviewed maar het is dus per ongeluk gesloten ipv dat het "approved" is.

@merlijn-jocque

merlijn-jocque commented Aug 3, 2026 via email

Copy link
Copy Markdown
Contributor

@Florian9041

Copy link
Copy Markdown
Contributor Author

@merlijn-jocque je zou nog een review moeten approven, zodat er rechtsboven een vinkje komt naast je naam (zoals bij Anouk). Dan kan Hans dit reviewen.

@Florian9041

Copy link
Copy Markdown
Contributor Author

Merci @merlijn-jocque!
@hansvancalster het staat klaar voor jouw review

@hansvancalster
hansvancalster merged commit fc11cd5 into main Aug 3, 2026
3 checks passed
@hansvancalster
hansvancalster deleted the sfp-101-nl branch August 3, 2026 11:52
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.

4 participants