Skip to content

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