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

Validation ds105 #4347

Merged
merged 9 commits into from
Jun 17, 2024
Merged

Validation ds105 #4347

merged 9 commits into from
Jun 17, 2024

Conversation

moishce
Copy link
Contributor

@moishce moishce commented Jun 17, 2024

@moishce moishce marked this pull request as ready for review June 17, 2024 08:51
Copy link

Changelog(s) in markdown:

  • Added DS105 to the new validation format. "check if DESCRIPTION file contains contribution details" #4347

Copy link
Contributor

@YuvHayun YuvHayun left a comment

Choose a reason for hiding this comment

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

Nice, minor comments.

.changelog/4347.yml Show resolved Hide resolved
class IsDescriptionContainsContribDetailsValidator(BaseValidator[ContentTypes]):
error_code = "DS105"
description = "check if DESCRIPTION file contains contribution details"
rationale = "the contribution/partner details will be generated automatically and we don't want should be duplicate"
Copy link
Contributor

Choose a reason for hiding this comment

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

rephrase it?

Given
content_items iterables.
- Case 1: description file without Contrib Details.
- Case 3: description file with Contrib Details.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Case 3: description file with Contrib Details.
- Case 2: description file with Contrib Details.

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