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

[Docs] jsx-first-prop-new-line: add missing multiprop value #3598

Merged
merged 1 commit into from
Jul 19, 2023

Conversation

dzek69
Copy link
Contributor

@dzek69 dzek69 commented Jul 7, 2023

Added missing possible value, The value is listed near the top of the docs document, but not near the bottom

Copy link
Member

@ljharb ljharb left a comment

Choose a reason for hiding this comment

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

Thanks!

@ljharb
Copy link
Member

ljharb commented Jul 7, 2023

Tests are failing on master rn, but I'll merge this once they're fixed.

@ljharb ljharb self-assigned this Jul 7, 2023
@ljharb ljharb changed the title Update jsx-first-prop-new-line.md [Docs] jsx-first-prop-new-line: add missing multiprop value Jul 19, 2023
@codecov
Copy link

codecov bot commented Jul 19, 2023

Codecov Report

Merging #3598 (4e72b82) into master (9c5ac98) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3598   +/-   ##
=======================================
  Coverage   97.62%   97.62%           
=======================================
  Files         132      132           
  Lines        9295     9295           
  Branches     3397     3397           
=======================================
  Hits         9074     9074           
  Misses        221      221           

@ljharb ljharb merged commit 4e72b82 into jsx-eslint:master Jul 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants