Skip to content

Conversation

artengin
Copy link
Contributor

Refs: #40

@coveralls
Copy link

coveralls commented Aug 28, 2025

Pull Request Test Coverage Report for Build 17732251598

Details

  • 29 of 30 (96.67%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.2%) to 98.639%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/Commands/InitCommand.php 28 29 96.55%
Totals Coverage Status
Change from base Build 17647802486: -0.2%
Covered Lines: 290
Relevant Lines: 294

💛 - Coveralls

@artengin artengin requested a review from DenTray August 28, 2025 18:37
@artengin artengin assigned AZabolotnikov and unassigned DenTray Sep 9, 2025
@artengin artengin assigned AZabolotnikov and unassigned artengin Sep 11, 2025
$config->write();
}

protected function addDefaultHttpExceptionRender(): void
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@DenTray

InitCommand class have 600+ strings. mb can we move this functionality in different class?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AZabolotnikov sure, but as a separatly described task :)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants