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

add r-fpp3 #27127

Merged
merged 2 commits into from
Aug 8, 2024
Merged

add r-fpp3 #27127

merged 2 commits into from
Aug 8, 2024

Conversation

cnrrobertson
Copy link
Contributor

used r skeleton helper script
@conda-forge-webservices
Copy link

Hi! This is the friendly automated conda-forge-linting service.

I wanted to let you know that I linted all conda-recipes in your PR (recipes/r-fpp3/meta.yaml) and found some lint.

Here's what I've got...

For recipes/r-fpp3/meta.yaml:

  • The following maintainers have not yet confirmed that they are willing to be listed here: conda-forge/r. Please ask them to comment on this PR if they are.

For recipes/r-fpp3/meta.yaml:

  • License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

Documentation on acceptable licenses can be found here.

Copy link
Member

@mfansler mfansler left a comment

Choose a reason for hiding this comment

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

Thanks for adding this! ❤️

Couple touch ups, but otherwise looks good.

recipes/r-fpp3/meta.yaml Outdated Show resolved Hide resolved
recipes/r-fpp3/meta.yaml Outdated Show resolved Hide resolved
Co-authored-by: Mervin Fansler <mmfansler@gmail.com>
@conda-forge-webservices
Copy link

Hi! This is the friendly automated conda-forge-linting service.

I wanted to let you know that I linted all conda-recipes in your PR (recipes/r-fpp3/meta.yaml) and found some lint.

Here's what I've got...

For recipes/r-fpp3/meta.yaml:

  • The following maintainers have not yet confirmed that they are willing to be listed here: conda-forge/r. Please ask them to comment on this PR if they are.

@mfansler mfansler merged commit 8df3fd7 into conda-forge:main Aug 8, 2024
2 of 5 checks passed
@mfansler
Copy link
Member

mfansler commented Aug 8, 2024

Looks good - thanks for adding! ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants