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

Added Code Climate badges #48

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

UlisesGascon
Copy link
Member

Description

Added Code Climate Badges

Related Issue

Related onebeyond/maintainers#113

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@CesarD
Copy link
Collaborator

CesarD commented Jul 15, 2023

Is there any way to configure the analysis?
Because I see many of the stuff mentioned on it is about unit tests and perhaps we should exclude them from the analysis.

@UlisesGascon
Copy link
Member Author

Is there any way to configure the analysis?

Yes, @CesarD.

You can change the rules (with some limitations), here

Captura de pantalla 2023-07-15 a las 14 10 42

Also you can manage individual issues status (so you can close or won't fix many of them too). In this View
Captura de pantalla 2023-07-15 a las 14 11 54

As well you can disable paths here:

Captura de pantalla 2023-07-15 a las 14 14 02

@UlisesGascon UlisesGascon requested a review from CesarD July 15, 2023 12:14
@CesarD
Copy link
Collaborator

CesarD commented Jul 15, 2023

Thanks Ulises!! I'll give it a more proper review later.
Have a nice weekend!

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