Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

feat: ✨ add cve validator#983

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

EduardoRMello
Copy link
Contributor

@EduardoRMelloEduardoRMello commentedAug 4, 2022•
edited
Loading

Fixes Or Enhances

Add validator to Common Vulnerabilities and Exposures Identifier according to mitre.org

Make sure that you've checked the boxes below before you submit PR:

  • Tests exist or have been written that cover this particular change.

@go-playground/validator-maintainers

@EduardoRMelloEduardoRMello requested a review froma team as acode ownerAugust 4, 2022 22:15
@coveralls
Copy link

coveralls commentedAug 4, 2022•
edited
Loading

Coverage Status

Coverage: 73.87% (+0.008%) from 73.862% when pulling7f01ebf on EduardoRMello:feat/add-cve-validator into58d420d on go-playground:master.

@EduardoRMello
Copy link
ContributorAuthor

Hello@deankarn ! Do you intend to maintain this repo ?

zemzale
zemzale previously approved these changesOct 5, 2022
Copy link
Member

@zemzalezemzale left a comment

Choose a reason for hiding this comment

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

Looks good to me. Has proper tests and seems to be in line with the spec.

@zemzale
Copy link
Member

@deankarn I don't have the right to merge/approve. Could you do it?

deankarn
deankarn previously approved these changesMar 19, 2023
@deankarndeankarn dismissed stale reviews fromzemzale and themself via7f01ebfMarch 19, 2023 23:51
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@deankarndeankarndeankarn approved these changes

@zemzalezemzalezemzale left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4 participants
@EduardoRMello@coveralls@zemzale@deankarn
[8]ページ先頭
©2009-2025 Movatter.jp