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

[NEW function] PCI2 #94

Closed
17 tasks done
RuttenAnneleen opened this issue Oct 29, 2024 · 0 comments · Fixed by #95
Closed
17 tasks done

[NEW function] PCI2 #94

RuttenAnneleen opened this issue Oct 29, 2024 · 0 comments · Fixed by #95
Assignees
Labels
Function Issue related to a function New Nieuwe functie/data

Comments

@RuttenAnneleen
Copy link
Contributor

RuttenAnneleen commented Oct 29, 2024

Voorstel functie naam "PCI2"

Functionality

Checklist

  • maak een nieuw R-bestand
  • vermijd library("package") en require("package") gebruik package::function() of als het echt niet anders gaat requirenamespace("package")
  • sla het R bestand op onder ./R met filenaam is gelijk aan functienaam
  • voorzie een functie titel met #' op regel 1 van je script
  • voorzie een auteur met #' @author
  • voorzie een beschrijving met #' @description
  • voorzie uitleg over de input parameter(s) met #' @param name
  • voorzie uitleg over de output van de functie met #' @returns
  • voorzie minstens 1 voorbeeld van het gebruik van de functie dmv #' @examples
  • voorzie de export - tag dmv #' @export (#14)
  • voer usethis::use_package("packagename", min_version = TRUE) uit in de console voor iedere package die je gebruikt.
  • voer roxygen2::roxygenise() uit in de console
  • voer devtools::check() uit in de console
  • los eventuele errors, warnings en notes1 op
  • maak een pull request met @soriadelva of @SanderDevisscher en eventueel andere relevante gebruikers als reviewer.

1in de mate van het mogelijke

@RuttenAnneleen RuttenAnneleen added Function Issue related to a function New Nieuwe functie/data labels Oct 29, 2024
SanderDevisscher added a commit that referenced this issue Nov 12, 2024
#94

@RuttenAnneleen I decided to move the entirety of the PCI R-script and convert them into seperate functions in fistools
SanderDevisscher added a commit that referenced this issue Nov 12, 2024
SanderDevisscher added a commit that referenced this issue Nov 13, 2024
SanderDevisscher added a commit that referenced this issue Nov 13, 2024
SanderDevisscher added a commit that referenced this issue Nov 13, 2024
SanderDevisscher added a commit that referenced this issue Nov 13, 2024
SanderDevisscher added a commit that referenced this issue Nov 13, 2024
SanderDevisscher added a commit that referenced this issue Nov 13, 2024
@SanderDevisscher SanderDevisscher linked a pull request Nov 13, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Function Issue related to a function New Nieuwe functie/data
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants