Skip to content

Conversation

@HanslettTheDev
Copy link

@HanslettTheDev HanslettTheDev commented Dec 19, 2025

This PR aims to close issue #3076 and continues from where Veeba left of

Changes made

@davidism
Copy link
Member

davidism commented Dec 19, 2025

Did you use an LLM while working on this? It appears you've changed the entire document, rather than only editing or adding. Your commit message only vaguely refers to editing and improvement. Please specifically say what the improvements are.

@HanslettTheDev
Copy link
Author

I actually wanted to rename the document and instead deleted the document.
I wanted to change the file name from into_to_command_line_tutorial.md to a file name consistently used in the docs directory ie command-line-tutorial using hyphens rather than underscores

@HanslettTheDev
Copy link
Author

Did you use an LLM while working on this? It appears you've changed the entire document, rather than only editing or adding. Your commit message only valet refers to editing and improvement. Please specifically say what the improvements are.

I actually indicated in the commit messages. Or you mean including it in the PR description?

@HanslettTheDev
Copy link
Author

I think I get your point now
I should have made the changes accordingly based on his already added commits, so the git diff can show exactly what I added and what I removed. Right now it looks like it's mostly my work in it(which is not the case)
My sincere apologies. Is it possible I can redo this PR in another similar branch so I can correct this properly?

@denuoweb
Copy link
Contributor

@HanslettTheDev Yes close this PR and open a new one with the history.

@HanslettTheDev
Copy link
Author

Got it @denuoweb

@HanslettTheDev HanslettTheDev deleted the docs-cli-tutorial branch December 20, 2025 13:33
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 4, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants