[go: nahoru, domu]

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

Port some templates to Twig #13207

Merged
merged 3 commits into from
Apr 28, 2017
Merged

Port some templates to Twig #13207

merged 3 commits into from
Apr 28, 2017

Conversation

MauricioFauth
Copy link
Member

Signed-off-by: Maurício Meneghini Fauth mauriciofauth@gmail.com

Before submitting pull request, please check that every commit:

  • Has proper Signed-Off-By
  • Has commit message which describes it
  • Is needed on it's own, if you have just minor fixes to previous commits, you can squash them
  • Any new functionality is covered by tests

Signed-off-by: Maurício Meneghini Fauth <mauriciofauth@gmail.com>
Signed-off-by: Maurício Meneghini Fauth <mauriciofauth@gmail.com>
@codecov
Copy link
codecov bot commented Apr 23, 2017

Codecov Report

Merging #13207 into master will decrease coverage by 0.14%.
The diff coverage is 95.45%.

@@            Coverage Diff             @@
##           master   #13207      +/-   ##
==========================================
- Coverage   54.55%    54.4%   -0.15%     
==========================================
  Files         469      470       +1     
  Lines       69553    69888     +335     
==========================================
+ Hits        37942    38023      +81     
- Misses      31611    31865     +254

Signed-off-by: Maurício Meneghini Fauth <mauriciofauth@gmail.com>
@nijel nijel self-assigned this Apr 28, 2017
@nijel nijel added this to the 4.8.0 milestone Apr 28, 2017
@nijel nijel merged commit 1b4b736 into phpmyadmin:master Apr 28, 2017
@nijel
Copy link
Contributor
nijel commented Apr 28, 2017

Thanks! While touching the tests, feel free to change their logic as well, the byte to byte verification of rendered HTML is not really optimal.

@MauricioFauth MauricioFauth deleted the templates branch April 29, 2017 23:34
@MauricioFauth MauricioFauth mentioned this pull request May 4, 2017
4 tasks
@MauricioFauth
Copy link
Member Author

Related to #13297

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.

None yet

2 participants