Skip to content

[docs]: Create README files for all folders#499

Open
sbreitbart-NOAA wants to merge 2 commits into
mainfrom
readmes
Open

[docs]: Create README files for all folders#499
sbreitbart-NOAA wants to merge 2 commits into
mainfrom
readmes

Conversation

@sbreitbart-NOAA

Copy link
Copy Markdown
Collaborator

Create README files for folders in the repo per #481

… add spaces next to bullets in pull request template to facilitate automatic formatting
@github-actions

Copy link
Copy Markdown
Contributor

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

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

@sbreitbart-NOAA sbreitbart-NOAA linked an issue Jun 16, 2026 that may be closed by this pull request
2 tasks
@github-actions

Copy link
Copy Markdown
Contributor

Code Metrics Report

Coverage Code to Test Ratio Test Execution Time
32.7% 1:0.2 24s

Reported by octocov


# What is the feature?
*
*

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
*
*

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Just removing the space bc its not neccessary


# How have you implemented the solution?
*
*

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
*
*


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

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
*
*

Comment thread .github/README.md
@@ -0,0 +1,8 @@
# GitHub Workflows and Templates

This folder contains the workflows to initiate:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I would add more information about the workflow. This is a folder housing GitHub workflows that perform actions such as X. I think it could be helpful for future us and maintainers to described how some of these were made (e.g. ghactions4r)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Why was this file deleted?

@Schiano-NOAA Schiano-NOAA left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This looks great! An awesome thing for future us, contributors, and long term maintenance for the package. I have a few questions and minor concerns with some suggestions. Let me know what you think! Once they are resolved, it should be good to go.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Why was this file deleted?

Comment thread inst/README.md

This folder contains various files which support the creation of materials for running functions in {asar}.

* WORDLIST contains a list of words that are used in the spell check workflow. It is a plain text file with one word per line. The words in this list are considered correct and will not be flagged as spelling errors during the spell check process.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* WORDLIST contains a list of words that are used in the spell check workflow. It is a plain text file with one word per line. The words in this list are considered correct and will not be flagged as spelling errors during the spell check process.
* WORDLIST contains a list of words that are used in the spell check workflow. It is a plain text file with one word per line. The words in this list are considered correct and will not be flagged as spelling errors during the spell check process. Words can be added or removed from this list by running `spelling::update_wordlist()`.

Comment thread inst/README.md

## Sub-folders

* The `extdata` sub-folder contains an example of an SS3 report file that can be input into `convert_output()`.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* The `extdata` sub-folder contains an example of an SS3 report file that can be input into `convert_output()`.
* The `extdata` sub-folder contains an example of an SS3 report file that can be input into `stockplotr::convert_output()`.

Comment thread R/README.md
@@ -0,0 +1,7 @@
# Functions

This folder is composed of all the exported and utility functions used in {asar}. These utility functions are typically used to perform intermediary tasks in the process of making a report, and are run within the other functions.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
This folder is composed of all the exported and utility functions used in {asar}. These utility functions are typically used to perform intermediary tasks in the process of making a report, and are run within the other functions.
This folder is composed of all the exported and utility functions (utils) used in {asar}. These utility functions are typically used to perform intermediary tasks in the process of making a report, and are run within the other functions.

Comment thread README.md

In this function, the users have the option to convert output files to a standardized
framework from either SS3 (Report.sso) or BAM (.rdat) output files. `asar` uses this
framework from either SS3 (Report.sso), BAM (.rdat), FIMS, or Rceattle output files. `asar` uses this

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
framework from either SS3 (Report.sso), BAM (.rdat), FIMS, or Rceattle output files. `asar` uses this
framework from either SS3 (Report.sso), BAM (.rdat), FIMS (R object), or Rceattle (R object) output files. `asar` uses this

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.

[docs] Create README files for each folder

2 participants