Skip to content

Add ruff lint workflow and address linting issues.#35

Merged
neelam-kushwah merged 2 commits intoeclipse-uprotocol:mainfrom
neelam-kushwah:main
Jun 6, 2024
Merged

Add ruff lint workflow and address linting issues.#35
neelam-kushwah merged 2 commits intoeclipse-uprotocol:mainfrom
neelam-kushwah:main

Conversation

@neelam-kushwah
Copy link
Contributor

@neelam-kushwah neelam-kushwah commented Jun 4, 2024

#36 , #14

Fix linting issues and refactor code for PEP 8 compliance ( except for the max length limit check, The PEP 8 guideline suggests a maximum line length of 79 characters for Python code, However, in practice, many projects and developers find this limit too restrictive for modern development environments. With wider screens and IDEs commonly setting default limits of 120 characters, it's common to see larger line length limits used in practice, typically around 100 or 120 characters. By allowing longer lines, code can often be more readable and maintainable, especially for function calls, method chains, or longer strings)

  • Organized imports in the correct order.
  • Refactored function and variable names to follow snake_case naming convention.
  • Ensured class names start with a capital letter.
  • Removed commented-out code to clean up the codebase.
  • Applied PEP 8 formatting rules to ensure consistency.
  • Addressed Pyflakes rules to fix any undefined names and other issues.

@neelam-kushwah neelam-kushwah added the enhancement New feature or request label Jun 4, 2024
@neelam-kushwah neelam-kushwah self-assigned this Jun 4, 2024
@github-actions
Copy link

github-actions bot commented Jun 4, 2024

Code coverage report is ready! 📈

@github-actions
Copy link

github-actions bot commented Jun 4, 2024

Code coverage report is ready! 📈

@github-actions
Copy link

github-actions bot commented Jun 6, 2024

Code coverage report is ready! 📈

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

Labels

enhancement New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Add GitHub workflow for linting and formatting Python code Add workflows for CodeQL Analysis and Lint Error Description

2 participants