This repository has been archived by the owner. It is now read-only.

added codacy config #20

Open
wants to merge 1 commit into
base: develop
from

Conversation

5 participants
@Fenn-CS
Copy link
Contributor

Fenn-CS commented Jun 26, 2018

Signed-off-by: Fenn-25 fenn25.fn@gmail.com

Description

Included .codacy.yml with a pylint engine, with an exclusion of tests from quality checker, we might want to move the test exclusion just to the level of coverage but more suggestions are needed on this.

Fixes #18

Type of Change:

  • Quality Assurance
  • Introduction of new tool

How Has This Been Tested?

Non code change, builds are unaffected, report from codacy observed.

Checklist:

  • My PR follows the style guidelines of this project
  • I have performed a self-review of my own code or materials
added codacy config
Signed-off-by: Fenn-25 <fenn25.fn@gmail.com>
@Fenn-CS

This comment has been minimized.

Copy link
Contributor

Fenn-CS commented Jul 8, 2018

Hello, @ExactlyMay @nida @MehaKaushik please this PR is lagging in reviews.

@Fenn-CS Fenn-CS requested review from nida , MayBurgos and MehaKaushik Jul 8, 2018

@nida

nida approved these changes Jul 13, 2018

@nida

nida approved these changes Jul 13, 2018

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