8000 [TwigBridge] Add `FormLayoutTestCase` class by ker0x · Pull Request #50127 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

[TwigBridge] Add FormLayoutTestCase class #50127

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. 10000 We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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!

@ker0x ker0x requested review from xabbuh and yceruto as code owners April 23, 2023 11:34
@carsonbot carsonbot added this to the 6.3 milestone Apr 23, 2023
@ker0x ker0x force-pushed the issue-49775 branch 2 times, most recently from 9d9c580 to c7275b9 Compare April 23, 2023 16:29
@@ -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

@nicolas-grekas nicolas-grekas modified the milestones: 6.3, 6.4 May 23, 2023
@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.

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
0