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

fix(attr-non-space-content-evaluate): Split no attribute and empty attribute message #2495

Merged
merged 1 commit into from
Aug 28, 2020

Conversation

xyc
Copy link
Contributor

@xyc xyc commented Aug 26, 2020

Split no attribute and empty attribute message

Closes issue: fixes #2451, fixes #2453

Reviewer checks

Required fields, to be filled out by PR reviewer(s)

  • Follows the commit message policy, appropriate for next version
  • Code is reviewed for security

@xyc xyc requested a review from a team as a code owner August 26, 2020 23:14
@straker
Copy link
Contributor

straker commented Aug 28, 2020

Sorry about the failing tests. We just merged a pr to fix them. Please merge latest develop into your branch to resolve the problems.

@xyc xyc force-pushed the xyc/split-no-attr-empty-attr branch from 4e865db to c1be0f6 Compare August 28, 2020 17:11
@xyc
Copy link
Contributor Author

xyc commented Aug 28, 2020

Thanks @straker! No worries - I've rebased on develop branch.

Copy link
Contributor

@straker straker left a comment

Choose a reason for hiding this comment

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

Great work! Thanks for putting this together.

@straker straker merged commit 5f822f4 into dequelabs:develop Aug 28, 2020
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.

Split the non-empty-alt fail message Split the non-empty-title fail message
2 participants