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

Fix test cran #108

Merged
merged 3 commits into from
Jan 16, 2024
Merged

Fix test cran #108

merged 3 commits into from
Jan 16, 2024

Conversation

MurielleDelmotte
Copy link
Contributor

@MurielleDelmotte MurielleDelmotte commented Jan 16, 2024

fix: test avec roxygen2 >=7.3.0

why:

  • with roxygen2 v7.3.0, tests fail (att_from_namespace)
  • roxygen2 now returns a message instead of a warning

what:

  • added a condition on the roxygen2 version to test a message or warning
  • check a minimum version for roxygen2 (>7.1.2)
  • removes testing on versions prior to 7.1.2

issue #107

why:
- with roxygen2 v7.3.0, tests fail (att_from_namespace)
- roxygen2 now returns a message instead of a warning

what:
- added a condition on the roxygen2 version to test a message or warning
- check a minimum version for roxygen2 (>7.1.2)
- removes testing on versions prior to 7.1.2

issue #107
why:
- missing contributing

what:
- add file CONTRIBUTING with the minimum package version {roxygen2)

issue #107
why:
- Complete documentation with modification

what:
- patch version package
- complete NEWS
- add language in DESCRIPTION

issue #107
@MurielleDelmotte MurielleDelmotte merged commit a4a3b8e into main Jan 16, 2024
9 checks passed
@MurielleDelmotte MurielleDelmotte deleted the fix_test_cran branch January 16, 2024 09:09
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.

1 participant