Skip to content

Various updates as described#446

Merged
Schiano-NOAA merged 8 commits intomainfrom
dev
Apr 15, 2026
Merged

Various updates as described#446
Schiano-NOAA merged 8 commits intomainfrom
dev

Conversation

@Schiano-NOAA
Copy link
Copy Markdown
Collaborator

What is the feature?

  • various updates pushed into dev. see release notes when merged

How have you implemented the solution?

Does the PR impact any other area of the project, maybe another repo?

sbreitbart-NOAA and others added 7 commits March 30, 2026 16:43
* Create snippets for custom figs/tables vignette; add to redesigned, less-redundant vignette

* Update snippet header formatting so tabs in main vignette work

* Update guidance to use gt package for tables

* Update vignette snippet about IDing table width/length and splitting

* Alter workflow so that user adds to existing figures/tables doc instead of creating new qmds; update workflow for adding figures/tables as qmds

* Update coding-in-qmd workflow

* Minor fixes

* Clarify vignette based on run-through

* Update vignettes/snippets/write-caps-alttext.Rmd

Co-authored-by: Sam Schiano <125507018+Schiano-NOAA@users.noreply.github.com>

* Update vignettes/snippets/save-caps-alttext.Rmd

Co-authored-by: Sam Schiano <125507018+Schiano-NOAA@users.noreply.github.com>

* Update vignettes/snippets/add-plots_rda.Rmd

Co-authored-by: Sam Schiano <125507018+Schiano-NOAA@users.noreply.github.com>

* Change header

* Resolve #443 (comment)

* Resolve several comments from review

* Resolve more comments from review

* add TOC to vignette

* Address last comment about markdown alt text/captions

---------

Co-authored-by: Sam Schiano <125507018+Schiano-NOAA@users.noreply.github.com>
* remove manual and link on site

* Update wordlist
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 15, 2026

Checklist

  • PR base branch is accurate
  • Is the code concise?
  • Comments are clear and useful.
  • Can you remove or combine any arguments?
  • Do argument contain defaults (if appliable)?
  • Code is documented and example provided (Roxygen).
  • Did you make a test (testthat)?
  • Was this tested under multiple scenarios?
  • Did you run devtools::check()?

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 15, 2026

New version checklist

  • Package version in DESCRIPTION has been updated
  • Release notes have been drafted/published
  • Cheatsheet content has been updated (if applicable)
  • Cheatsheet version has been updated

@github-actions
Copy link
Copy Markdown
Contributor

Code Metrics Report

Coverage Code to Test Ratio Test Execution Time
33.8% 1:0.2 25s

Reported by octocov

@Schiano-NOAA Schiano-NOAA merged commit 3f82283 into main Apr 15, 2026
9 checks passed
@Schiano-NOAA Schiano-NOAA deleted the dev branch April 15, 2026 21:25
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