View GitHub Repository smr

Schedule Inspection
Subscribe

Scrutinizer GitHub App not installed

We could not synchronize checks via GitHub's checks API since Scrutinizer's GitHub App is not installed for this repository.

Install GitHub App

Failed Conditions
Pull Request — master (#1094)
by Dan
05:37
created

Changes

1 failure condition met
issues.severity(>= INFO).new.exists
62 issues (8 major, 54 minor) were introduced.
3 issues were fixed.
10 new patches (Indentation, Spacing, Braces) are available.
44 added classes/operations
Arrow A AbstractSmrPlayer::getUnvisitedSectors() added
Arrow A Database::reconnect() added
Arrow A Database::read() added
Arrow A Database::write() added
Arrow A DatabaseResult::getNumRecords() added
Arrow A DatabaseResult::__construct() added
Arrow A DatabaseResult::records() added
Arrow A DatabaseResult::hasRecord() added
Arrow A DatabaseResult::record() added
Arrow A DatabaseRecord::getBoolean() added
F ↗ A Database improved
B ↗ A notice_nickserv_registered_user() improved
B ↗ A main_page() improved
B ↗ A SmrSector::__construct() improved
A ↘ B checkForAttackMessage() got worse
view all  
5.63
▲   + 0.07
Head
Dan Hemberger  
Merge d32176e14e608f21832c3377afd675145d373b18 into f174b55ac85cb70dd5b030d141837f57aaf789e5
⋮ 2 more commits
Base
Dan Hemberger  
Merge pull request #1093 from hemberger/typehints