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

Added FullText #13303 #13716

Merged
merged 1 commit into from Oct 2, 2017

Conversation

Projects
None yet
2 participants
@ankitjain28may
Contributor

ankitjain28may commented Sep 30, 2017

Signed-off-by: Ankit Jain ankitjain28may77@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
@codecov

This comment has been minimized.

Show comment
Hide comment
@codecov

codecov bot Sep 30, 2017

Codecov Report

Merging #13716 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #13716   +/-   ##
=======================================
  Coverage   53.81%   53.81%           
=======================================
  Files         481      481           
  Lines       82181    82181           
=======================================
  Hits        44228    44228           
  Misses      37953    37953

codecov bot commented Sep 30, 2017

Codecov Report

Merging #13716 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #13716   +/-   ##
=======================================
  Coverage   53.81%   53.81%           
=======================================
  Files         481      481           
  Lines       82181    82181           
=======================================
  Hits        44228    44228           
  Misses      37953    37953
Added FullText #13303
Signed-off-by: Ankit Jain <ankitjain28may77@gmail.com>

@nijel nijel self-assigned this Oct 2, 2017

@nijel nijel added this to the 4.8.0 milestone Oct 2, 2017

@nijel nijel merged commit 15e16cd into phpmyadmin:master Oct 2, 2017

5 of 6 checks passed

DCO Expected "ankitjain28may <ankitjain28may77@gmail.com>", but got "Ankit Jain <ankitjain28may77@gmail.com>"
Details
Scrutinizer Analysis: No new issues – Tests: passed
Details
codacy/pr Good work! A positive pull request.
Details
codecov/patch Coverage not affected when comparing f8bd078...15e16cd
Details
codecov/project 53.81% remains the same compared to f8bd078
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

nijel added a commit that referenced this pull request Oct 2, 2017

Changelog for #13716 and #13303
Signed-off-by: Michal Čihař <michal@cihar.com>
@nijel

This comment has been minimized.

Show comment
Hide comment
@nijel

nijel Oct 2, 2017

Member

Merged, thanks for your contribution!

Member

nijel commented Oct 2, 2017

Merged, thanks for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment