Skip to content

feat: add debug option to CLI - #134

Closed
jedluhmann wants to merge 0 commit into
andreyvit:masterfrom
jedluhmann:jl-add-debug-feature
Closed

jedluhmann wants to merge 0 commit into
andreyvit:masterfrom
jedluhmann:jl-add-debug-feature

Conversation

@jedluhmann

Copy link
Copy Markdown

Added a new debug feature. See the following commits for details:

@jedluhmann

jedluhmann commented Sep 5, 2026

Copy link
Copy Markdown
Author

@andreyvit I don't understand why you closed PR #135 basically saying that patch was too big. That PR only changes one file, adding just one test.

All of my PRs build on top of the previous one, so the latter ones appear to have a lot of unrelated changes, but that is not really the case. I've put a lot of effort into explaining my work to make it easier for you to review the changes and I've spent a lot of time updating the test cases, so you can have confidence in my work.

As far as styling my code like the rest of the project, I'm using ESLint to ensure that my code is formatted according to the industry wide standard. Is it just a matter of removing semicolons? I'm happy to do that if that's what you want.

@jedluhmann jedluhmann closed this Sep 12, 2026
@jedluhmann
jedluhmann deleted the jl-add-debug-feature branch September 12, 2026 03:20
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.

1 participant