Skip to content

remove return(x) from end of function#124

Merged
sbreitbart-NOAA merged 1 commit intonmfs-ost:mainfrom
Melissa-Karp:main
Aug 6, 2025
Merged

remove return(x) from end of function#124
sbreitbart-NOAA merged 1 commit intonmfs-ost:mainfrom
Melissa-Karp:main

Conversation

@Melissa-Karp
Copy link
Copy Markdown
Contributor

Addresses issue #116 to remove last return statements.

Co-authored-by: Bai Li - NOAA <Bai-Li-NOAA@users.noreply.github.com>
Co-authored-by: Andrea-Havron-NOAA <Andrea-Havron-NOAA@users.noreply.github.com>
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@sbreitbart-NOAA sbreitbart-NOAA self-requested a review August 4, 2025 15:43
@sbreitbart-NOAA
Copy link
Copy Markdown
Collaborator

spelling::spell_check_package() output: No spelling errors found.

@github-actions

This comment has been minimized.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Aug 6, 2025

Code Metrics Report

Coverage Code to Test Ratio Test Execution Time
62.7% 1:0.2 1m26s

Code coverage of files in pull request scope (66.7%)

Files Coverage
R/extract_caps_alttext.R 100.0%
R/plot_abundance_at_age.R 26.1%
R/plot_biomass.R 91.2%
R/plot_biomass_at_age.R 0.0%
R/plot_indices.R 0.0%
R/plot_landings.R 82.2%
R/plot_recruitment.R 97.0%
R/plot_recruitment_deviations.R 94.1%
R/plot_spawn_recruitment.R 100.0%
R/plot_spawning_biomass.R 90.4%
R/ss3_utils.R 0.0%
R/table_bnc.R 93.1%
R/table_indices.R 86.0%
R/table_landings.R 91.4%

Reported by octocov

@sbreitbart-NOAA sbreitbart-NOAA merged commit 1e0b127 into nmfs-ost:main Aug 6, 2025
6 of 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