Skip to content

ADD CONTAINS#7

Merged
l0rd merged 1 commit into
l0rd:masterfrom
xinity:check_contains
Jan 11, 2016
Merged

ADD CONTAINS#7
l0rd merged 1 commit into
l0rd:masterfrom
xinity:check_contains

Conversation

@xinity

@xinity xinity commented Jan 3, 2016

Copy link
Copy Markdown

No description provided.

@l0rd

l0rd commented Jan 6, 2016

Copy link
Copy Markdown
Owner

This PR include also the commits of PR #6. Can you please rebase this branch with master ?

@xinity

xinity commented Jan 8, 2016

Copy link
Copy Markdown
Author

almost finished :)
need to get ride of "grep -q ! " instead of "! grep -q" and will be fine

@l0rd

l0rd commented Jan 8, 2016

Copy link
Copy Markdown
Owner

Great! Thanks @xinity

@xinity

xinity commented Jan 11, 2016

Copy link
Copy Markdown
Author

all right, all fixed except i haven't found how to get rid of "grep -q !" to make it "! grep -q''

i'll update this PR anyway, if you have any clues please let me know.

sorry not having not enough GO skills :(

@l0rd

l0rd commented Jan 11, 2016

Copy link
Copy Markdown
Owner

No problem for the grep -q ! I will fix it in a separate commit. Thanks for your contribution @xinity!

l0rd added a commit that referenced this pull request Jan 11, 2016
Implement FILE_CONTAINS template
@l0rd l0rd merged commit 8620c02 into l0rd:master Jan 11, 2016
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