Skip to content

Extend verify target in Makefile#8

Merged
tisnik merged 1 commit intolightspeed-core:mainfrom
lpiwowar:feature/add-radon-target
Jun 2, 2025
Merged

Extend verify target in Makefile#8
tisnik merged 1 commit intolightspeed-core:mainfrom
lpiwowar:feature/add-radon-target

Conversation

@lpiwowar
Copy link
Copy Markdown
Contributor

@lpiwowar lpiwowar commented Jun 2, 2025

This commit:

  • Adds check-code-metrics target to Makefile to allow the users of the code to easily check the code with Radon prior to pushing to the repo. The code for the target is borrowed from [1].

  • Adds check-coverage target that runs the unittests and computes the coverage.

[1] https://github.com/davidslusser/actions_python_radon/blob/main/radon.sh

This commit:

 - Adds check-code-metrics target to Makefile to allow the users of the
   code to easily check the code with Radon prior to pushing to the repo.
   The code for the target is borrowed from [1].

 - Adds check-coverage target that runs the unittests and computes the
   coverage.

[1] https://github.com/davidslusser/actions_python_radon/blob/main/radon.sh
@lpiwowar lpiwowar marked this pull request as ready for review June 2, 2025 12:14
Copy link
Copy Markdown
Collaborator

@tisnik tisnik left a comment

Choose a reason for hiding this comment

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

approved

@tisnik tisnik merged commit 1f73002 into lightspeed-core:main Jun 2, 2025
8 checks passed
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.

2 participants