Skip to content

Conversation

@auzaheta
Copy link
Collaborator

Description

Patch solving notes from CRAN checks

Checklist:

  • I have added tests that prove my fix is effective or that my feature works
  • The package builds on my OS without issues (MacOS Sequoia 15.4)
  • My changes generate no new warnings
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (.R, NEWS.md)
  • I have bumped the version by the appropriate increment in the DESCRIPTION file (major, minor, patch)

@auzaheta auzaheta requested a review from Copilot April 13, 2025 00:07
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 22 out of 37 changed files in this pull request and generated 2 comments.

Files not reviewed (15)
  • CRAN-SUBMISSION: Language not supported
  • DESCRIPTION: Language not supported
  • NAMESPACE: Language not supported
  • R/functions_checks.R: Language not supported
  • R/functions_effects_DyNAM_choice.R: Language not supported
  • R/functions_effects_DyNAM_rate.R: Language not supported
  • R/functions_effects_DyNAMi_choice.R: Language not supported
  • R/functions_effects_DyNAMi_rate.R: Language not supported
  • R/functions_effects_REM.R: Language not supported
  • README.Rmd: Language not supported
  • cleanup: Language not supported
  • cleanup.win: Language not supported
  • configure.ac: Language not supported
  • inst/REFERENCES.bib: Language not supported
  • src/Makevars.in: Language not supported
Comments suppressed due to low confidence (1)

src/gather_progress.cpp:3

  • Verify that the change from '.hpp' to '.h' is consistent with the project's header naming conventions to avoid potential include mismatches.
#include "gather_progress.h"

@auzaheta auzaheta merged commit c3d5aa3 into main Apr 13, 2025
8 checks passed
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.

2 participants