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

Package validator cpmr0020 flagged when nuspec does not contain email. #200

Open
TheCakeIsNaOH opened this issue Oct 21, 2022 · 8 comments

Comments

@TheCakeIsNaOH
Copy link
Member

TheCakeIsNaOH commented Oct 21, 2022

What You Are Seeing?

For plexamp, the cpmr0020 rule is flagged, which is indicating that nuspec contains an email. However, the nuspec does not contain an email address.

It is likely flagging because the description contains this url: https://cdn-images-1.medium.com/max/2000/1*[email protected]

What is Expected?

That the rule is not flagged.

How Did You Get This To Happen? (Steps to Reproduce)

N/A

System Details

N/A

Output Log

N/A

┆Issue is synchronized with this Gitlab issue by Unito

@TheCakeIsNaOH
Copy link
Member Author

@TheCakeIsNaOH
Copy link
Member Author

@flcdrg
Copy link
Member

flcdrg commented Jan 27, 2023

@flcdrg
Copy link
Member

flcdrg commented Jan 27, 2023

@Windos
Copy link
Member

Windos commented May 10, 2023

https://community.chocolatey.org/packages/qbs/2.0.0

This one does not have any email addresses in the author or copyright fields, though it does in the release notes so it could be triggering against that.

@TheCakeIsNaOH
Copy link
Member Author

@Windos That check was recently expanded to cover all the nuspec fields, but the message was not updated to reflect that change.
#187

The package does have emails in the release notes, so the rule is triggering correctly in this case, and is not hitting this bug.
image

@Windos
Copy link
Member

Windos commented May 10, 2023

Ahh, I hadn't picked up on that change

@Thilas
Copy link

Thilas commented Oct 7, 2023

https://community.chocolatey.org/packages/StrawberryPerl/5.38.0: there's is something that looks like an email in the mailingListUrl but this is a valid url anyway (as a workaround, I'll encode the @ for the time being).

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

No branches or pull requests

5 participants