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

[TwigBridge] Add FormLayoutTestCase class #50127

Merged
merged 1 commit into from Oct 2, 2023
Merged

Conversation

ker0x
Copy link
Contributor

@ker0x ker0x commented Apr 23, 2023

Q A
Branch? 6.4
Bug fix? no
New feature? yes
Deprecations? no
Tickets Fix #49775
License MIT
Doc PR TODO

Would require #50126 to be merged before!

@@ -61,46 +60,6 @@ protected function tearDown(): void
parent::tearDown();
}

protected function assertXpathNodeValue(\DOMElement $element, $expression, $nodeValue)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I removed this method because it was never used

@ker0x
Copy link
Contributor Author

ker0x commented Jul 26, 2023

Failing test seems to be unrelated with these changes!

@fabpot
Copy link
Member

fabpot commented Oct 1, 2023

@ker0x Can you rebase on current 6.4 and resolve the conflict? Thank you.

@ker0x
Copy link
Contributor Author

ker0x commented Oct 2, 2023

@fabpot It's done.

@fabpot
Copy link
Member

fabpot commented Oct 2, 2023

Thank you @ker0x.

@fabpot fabpot merged commit 19f0093 into symfony:6.4 Oct 2, 2023
4 of 9 checks passed
@ker0x ker0x deleted the issue-49775 branch October 2, 2023 06:47
This was referenced Oct 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[RFC] Simplify form theme testing for external packages
4 participants