-
Notifications
You must be signed in to change notification settings - Fork 823
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
DEP Replace thirdparty simpletest with symfony domcrawler #10643
DEP Replace thirdparty simpletest with symfony domcrawler #10643
Conversation
We should run this through a full kitchen sink test suite before considering it safe |
c35e739
to
2d83487
Compare
2d83487
to
3815e1a
Compare
3815e1a
to
d23191b
Compare
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.
A few small changes here to be made
2062ea5
to
5d9190f
Compare
c85f588
to
d8aa323
Compare
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.
Nice! Just one very very tiny change to make.
0c72686
to
2e08a90
Compare
Co-authored-by: Guy Sartorelli <[email protected]>
2e08a90
to
9d13f35
Compare
I can't click "approve" because I created the PR originally but I approve. |
Proof of concept for replacing the
simpletest
with symfony/domcrawlerParent issue