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

Debug flag added #30

Merged
merged 4 commits into from
Nov 24, 2024
Merged

Debug flag added #30

merged 4 commits into from
Nov 24, 2024

Conversation

Ishan-Jadhav
Copy link
Contributor

I've added the --debug flag to the implementation. Could you please review and confirm if this aligns with what was required? Let me know if any further adjustments or changes are needed.

@Veinar
Copy link
Owner

Veinar commented Nov 24, 2024

Hi @Ishan-Jadhav, thanks for your time and work. I'll look at this in couple of hours. But for a first sight it looks fantastic.

@Veinar Veinar self-assigned this Nov 24, 2024
@Veinar
Copy link
Owner

Veinar commented Nov 24, 2024

@Ishan-Jadhav I've added some commits for your forked repo (black formatting and pylint output suggestions) please update your PR with those commits. And yes, about task you've implemented outstanding work, thank you 🙏

@Veinar Veinar merged commit 27143be into Veinar:develop Nov 24, 2024
3 checks passed
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