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

Feat: Add Linting and Code Quality Checks in CI #32

Closed
noble-varghese opened this issue Oct 10, 2023 · 0 comments · Fixed by #41
Closed

Feat: Add Linting and Code Quality Checks in CI #32

noble-varghese opened this issue Oct 10, 2023 · 0 comments · Fixed by #41
Labels
continuous-integration enhancement New feature or request good first issue Good for newcomers

Comments

@noble-varghese
Copy link
Contributor

Description:

Maintaining code quality and adherence to coding standards is essential for the long-term stability and maintainability of our project. While we have some development checks in place, enhancing our Continuous Integration (CI) pipeline with linting and code quality checks can help ensure that our codebase remains clean and consistent. These checks will analyze our codebase for potential issues, such as coding style violations and common errors, helping us catch and address them early in the development process.

Proposed Tasks:

  • Integrate a linting tool (Pylint) into our CI pipeline.
  • Configure linting rules to match our coding standards.
  • Set up code quality checks (e.g., code complexity analysis) as needed.
  • Ensure that the CI pipeline fails on linting errors.
  • Document linting and code quality procedures for the development team.

Code quality is an ongoing effort; new code submissions should adhere to coding standards.
We welcome community contributions and input on linting rules.

Note: This issue tracks the addition of linting and code quality checks to our CI pipeline. Further discussions and contributions will happen in follow-up discussions and pull requests.

Your involvement in this issue, whether through code contributions, linting rule suggestions, or feedback, is vital as we work to improve code quality in our project.

@noble-varghese noble-varghese changed the title SDK: Add CI tests for lint. checks Feat: Add Linting and Code Quality Checks in CI Oct 10, 2023
@noble-varghese noble-varghese added enhancement New feature or request good first issue Good for newcomers continuous-integration labels Oct 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
continuous-integration enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant