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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove jsonsempai. #997
Remove jsonsempai. #997
Conversation
9a78c67
to
d96c537
Compare
aquarius/events/events_monitor.py
Outdated
@@ -571,7 +565,7 @@ def get_and_process_event_logs_for_one_block(self, block): | |||
try: | |||
logs = self._web3.eth.get_logs(filter_params) | |||
self.process_logs(logs, block) | |||
except Exception as e: | |||
except Exception: |
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.
should we keep the error and print it if possible?
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.
there's already an explicit error logged. I can print out the text if you think it's needed. I just removed the unused variable based on pre-commit issues, but let me know
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.
yes please, print the e
message.
263164f
to
94efc94
Compare
Code Climate has analyzed commit 5c320aa and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 93.1% (50% is the threshold). This pull request will bring the total coverage in the repository to 84.5%. View more on Code Climate. |
No description provided.