-
Notifications
You must be signed in to change notification settings - Fork 337
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
Autocomplete and Date Picker Combobox Examples: Update to conform with latest APG coding guidelines #1623
Conversation
Regression test coverage:Examples without any regression tests:
Examples missing some regression tests:
Example pages with Keyboard or Attribute table rows that do not have data-test-ids:
SUMMARY:55 example pages found. ERROR - missing tests: Please write missing tests for this report to pass. |
I looks like the merge conflicts are only related to the result of the linter running on the code. So you can resolve the code conflicts by checking out your copy of the file during the merge:
I'm not sure why we are seeing the error in github actions above, but I'm looking into it! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These changes look good and the example works as expected -- the merge conflicts just need to be addressed.
I can run npm regression-report
locally on this branch and it succeeded, so I don't think the failure in the github checks comment here is related to that.
Thank you @jongund |
Since this is an important example in the APG, I thought it would be important to update the code to the new coding practices to make it easier to read.