Skip to content

Lint fmt#336

Merged
Avdhesh-Varshney merged 3 commits intocode-a2z:mainfrom
Abeeujah:lint-fmt
Oct 6, 2025
Merged

Lint fmt#336
Avdhesh-Varshney merged 3 commits intocode-a2z:mainfrom
Abeeujah:lint-fmt

Conversation

@Abeeujah
Copy link
Contributor

@Abeeujah Abeeujah commented Oct 4, 2025

Closes: #297

Describe the add-ons or changes you've made 📃

Perusing through the issue description and previous PRs raised to address, a requirement stood out, Focus on 2-space separated indentation in python files. Black, is an opinionated, no config formatter, on the other hand ruff is flexible, but achieves good and loved defaults out of the box.

The project can benefit from the uv ecosystem, not just from ruff, by using uv, the steps/moving pieces to reason about when setting the project up as a user or contributor shrinks, as uv by default again handles a lot of the primitives you'd have to worry about, including setting up a virtual environment.

Working Video 📷

Screencast_20251004_220652.webm

Type of change ☑️

What sort of change have you made:

  • Bug fix
  • New feature
  • Code style update (formatting, local variables)
  • Documentation update
  • Database update

Note to reviewers 📄

@github-actions
Copy link

github-actions bot commented Oct 4, 2025

Thank you for submitting your pull request! We'll review it as soon as possible. For further communication, join our discord server https://discord.gg/tSqtvHUJzE.

@Avdhesh-Varshney
Copy link
Collaborator

Hi @Abeeujah
I've a question, using uv the other contributors formatting will be setup automatically to be 2-space indentation?
Or need a manual command?

@Avdhesh-Varshney Avdhesh-Varshney added the question❓ Further information is requested label Oct 5, 2025
@Abeeujah
Copy link
Contributor Author

Abeeujah commented Oct 5, 2025

Automatically, I already configured the formatter to use 2 spaces indent width

indent-width = 2

Copy link
Collaborator

@Avdhesh-Varshney Avdhesh-Varshney left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, Approved.

@Avdhesh-Varshney Avdhesh-Varshney merged commit 4983a9a into code-a2z:main Oct 6, 2025
2 checks passed
@Avdhesh-Varshney Avdhesh-Varshney added hacktoberfest-accepted onlydust-wave Contribute to awesome OSS repos during OnlyDust's open source week and removed question❓ Further information is requested labels Oct 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest-accepted onlydust-wave Contribute to awesome OSS repos during OnlyDust's open source week

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add CI Workflow for Linting and Code Quality Checks

2 participants