Skip to content

Add new vue/require-prop-comment rule #2018

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

Closed
wants to merge 0 commits into from

Conversation

czb3279338858
Copy link
Contributor

@czb3279338858 czb3279338858 commented Oct 24, 2022

Fixes #1993

@czb3279338858
Copy link
Contributor Author

czb3279338858 commented Oct 24, 2022

npm run update,Changed the description of many files,I'm not sure this is correct, but I still submitted the code

@czb3279338858
Copy link
Contributor Author

1 failing:The file causing the error is not modified by me. Should I repair it before submitting

@FloEdelmann FloEdelmann changed the title add require-prop-comment rule Add new vue/require-prop-comment rule Oct 24, 2022
@czb3279338858
Copy link
Contributor Author

Where there is dispute, should I initiate a new merger request? @FloEdelmann

@FloEdelmann
Copy link
Member

You don't need to open a new PR, you can instead push new commits to this branch. Also, rebasing and force-pushing is not required, we will squash the whole branch into one commit on the master branch when merging the PR.

@czb3279338858
Copy link
Contributor Author

@FloEdelmann I push new commits.

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.

require-prop-comment
2 participants