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

Rename KEEP to KEEP_KEYS #223

Merged
merged 1 commit into from
Feb 2, 2019
Merged

Conversation

3timeslazy
Copy link
Member

I found collision between lexer and object function. This is my version of fix :)

@codecov
Copy link

codecov bot commented Jan 17, 2019

Codecov Report

Merging #223 into master will not change coverage.
The diff coverage is 50%.

@@          Coverage Diff           @@
##           master    #223   +/-   ##
======================================
  Coverage    59.2%   59.2%           
======================================
  Files         180     180           
  Lines        5249    5249           
======================================
  Hits         3106    3106           
  Misses       1799    1799           
  Partials      344     344

@ziflex ziflex merged commit 2451271 into MontFerret:master Feb 2, 2019
@ziflex
Copy link
Member

ziflex commented Feb 2, 2019

Great! Thank you for the PR!

@3timeslazy 3timeslazy deleted the rename_keep branch April 10, 2019 09:57
3timeslazy pushed a commit to 3timeslazy/ferret that referenced this pull request Apr 10, 2019
3timeslazy pushed a commit to 3timeslazy/ferret that referenced this pull request Apr 10, 2019
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.

2 participants