-
-
Notifications
You must be signed in to change notification settings - Fork 253
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
Add Golang package #348
Merged
Merged
Add Golang package #348
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
f440902
fix typos in dates in JSON
starius fb4ca37
add Go package
starius 1e03d93
README: update Go instructions
starius 099a79d
golang: add benchmark
starius eb84428
golang: use go-re2 for regular expresion matching
starius b741c0b
add github workflow for Go
starius eacd6fd
golang: add benchmark for MatchingCrawlers
starius 65e66d1
golang: speed-up MatchingCrawlers
starius 51acf89
README: instruct to install C++ RE2
starius 3f3adbf
fix link in README
starius 18e9e1b
Merge branch 'master' into golang
monperrus ecb680d
Merge branch 'master' into golang
monperrus 9262954
simplify CI
monperrus 473be9d
simplify CI
monperrus ee4872b
explicit validation
monperrus cc59a03
golang: println(pattern), use pattern as subtest
starius c402815
README: add example of Go program
starius 0bc397e
golang: remove copy-paste from benchmark test
starius 9fc7a2e
golang: benchmark on browser UA
starius 266245e
golang: don't use stretchr/testify
starius f08c3de
golang: check results in banchmark, test FP
starius a32a149
golang: switch back to standard Go Regexp
starius 2945d30
golang: add test against false negatives
starius File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use fmt.Println to print each pattern on a separate line.
Also maybe it is better to put crawler.Pattern as subtest name (first argument of t
.Run
) and run withgo test -v
, it will print each subtest name (which would be a pattern).There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great idea, could you do it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done. I pushed to the branch.