Skip to content
This repository was archived by the owner on Jan 9, 2023. It is now read-only.

Conversation

@hlimbo
Copy link

@hlimbo hlimbo commented Jan 31, 2020

  • requesting to rename expectLog function to expectLogContains for better readability
  • removed extra parameter condition not being actively used

@ghost
Copy link

ghost commented Jan 31, 2020

DeepCode's analysis on #79d741 found:

  • 0 critical issues. ⚠️ 0 warnings and 0 minor issues. ✔️ 0 issues were fixed.

💬 This comment has been generated by the DeepCode bot, installed by the owner of the repository. The DeepCode bot protects your repository by detecting and commenting on security vulnerabilities or other critical issues.


☺️ If you want to provide feedback on our bot, here is how to contact us.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 95.003% when pulling 79d741e on hlimbo:test/test_utils into c70aedf on Unity-Technologies:master.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants