Skip to content
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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: add return statements configurations into code climate file #61

Merged
merged 1 commit into from
May 14, 2024

Conversation

EricNiyo73
Copy link
Collaborator

@EricNiyo73 EricNiyo73 commented May 13, 2024

What does this PR do?
This PR adds configuration for the return-statements check in the codeclimate.yml file
Description of Task to be completed?
We're enabling a check that makes sure our code doesn't have too many return statements in methods. This helps keep our code clean and easy to understand.

The fix involves enabling the return-statements check and setting a threshold of 50 in the codeclimate.yml file.

Copy link

render bot commented May 13, 2024

@coveralls
Copy link

coveralls commented May 13, 2024

Coverage Status

coverage: 96.54%. remained the same
when pulling 7bbd37d on fx-add-return-statements
into 40770ff on develop.

@coveralls
Copy link

Coverage Status

coverage: 96.54%. remained the same
when pulling e5ee82c on fx-add-return-statements
into 40770ff on develop.

@EricNiyo73 EricNiyo73 changed the title Fix: add return statements configurations into code climate file. Fix: add return statements configurations into code climate file May 13, 2024
Copy link

codeclimate bot commented May 13, 2024

Code Climate has analyzed commit 7bbd37d and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 97.8% (0.0% change).

View more on Code Climate.

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

Successfully merging this pull request may close these issues.

None yet

4 participants