Skip to content

Release 1.0.1#216

Merged
DarianGill merged 5 commits into
mainfrom
release-1.0.1
Mar 31, 2026
Merged

Release 1.0.1#216
DarianGill merged 5 commits into
mainfrom
release-1.0.1

Conversation

@DarianGill

Copy link
Copy Markdown
Collaborator

Deployed to https://dev.vegbank.org/rc/ and https://dev.vegbank.org/beta/

What

A patch release that updates vegbankr dependency to 1.0.0 from 0.9.0

Testing and Docs

All 1577 tests still pass, and after updating R, vegbankr, and vegbankweb, check() only notes:

❯ checking dependencies in R code ... NOTE
Namespace in Imports field not imported from: ‘markdown’
All declared Imports should be used.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Patch release updating vegbankweb metadata for the 1.0.1 release, primarily to bump the vegbankr dependency to the 1.0.0 series.

Changes:

  • Bumped package version/date to 1.0.1 / 2026-03-31.
  • Updated dependency requirement to vegbankr (>= 1.0.0).
  • Updated RoxygenNote to 7.3.3.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread DESCRIPTION
@DarianGill DarianGill requested a review from mbjones March 31, 2026 20:25

@mbjones mbjones left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This all looks great. I tracked down a few additional syntax issues to fix the CRAN check NOTES that were being generated, so now it builds totally cleanly. The things I fixed:

  • quoted acronyms in DESCRIPTION
  • added spelling list to DESCRIPTION
  • depend on R >= 4.1.0 due to pipe operator |>
  • Fix import for markdown package
  • remove LazyData from DESCRIPTION that generates note
  • Fix syntax of @family tag in run_app() where a plain text string is required (no markdown)

Seems ready to release to me.

@DarianGill DarianGill merged commit 7c86fbb into main Mar 31, 2026
2 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.

3 participants