-
Notifications
You must be signed in to change notification settings - Fork 172
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
Dig profiler #253
Dig profiler #253
Conversation
@glaslos you were right with #237 (comment) |
|
ping ? |
@katkad can you briefly describe this feature? |
There are also some conflicts from the other pull requests I merged. |
hi, it is #237 but tests for this PR pass |
Removed leftovers Added profiler section
22ae9ff
to
8ef910e
Compare
rebased. but I don't understand errors from Travis |
|
Ah, the test failed because of some changes I did in my backend. Should run through fine now. |
@katkad test passed. Feel free to squash and merge. |
finaly in master f1842ba |
test PR, to see if dig is missing
edit: also do not shorten error message so we can get more than:
"Error inserting attack event into main database: (OperationalError) table events has"