Skip to content

Add TitleCommand to set terminal window title#17

Merged
anshumanjadiya1102 merged 4 commits intoDrive-for-Java:mainfrom
nihaltp:feat/title
Oct 12, 2025
Merged

Add TitleCommand to set terminal window title#17
anshumanjadiya1102 merged 4 commits intoDrive-for-Java:mainfrom
nihaltp:feat/title

Conversation

@nihaltp
Copy link
Contributor

@nihaltp nihaltp commented Oct 11, 2025

See #13

@nihaltp
Copy link
Contributor Author

nihaltp commented Oct 12, 2025

@anshumanjadiya1102 fixed

@anshumanjadiya1102 anshumanjadiya1102 added hacktoberfest-accepted This is for Hacktoberfest hacktoberfest This is for Hacktoberfest labels Oct 12, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements a new terminal command to set the window title dynamically. The implementation adds a TitleCommand class that uses ANSI escape sequences to change the terminal window title and registers it as the "title" command in the shell.

  • Adds a new TitleCommand class that implements the Command interface
  • Registers the "title" command in the main application's command registry

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
src/main/java/com/mycmd/commands/TitleCommand.java New command implementation that sets terminal window title using ANSI escape sequences
src/main/java/com/mycmd/App.java Registers the new title command in the command registry

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@anshumanjadiya1102
Copy link
Member

@nihaltp, Please check above suggestions

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@nihaltp
Copy link
Contributor Author

nihaltp commented Oct 12, 2025

@anshumanjadiya1102 added the suggestion.
i was copying how cmd works, that is why i didn't add anything when title was called without text. should i add feedback for future commands

@anshumanjadiya1102
Copy link
Member

Yes @nihaltp, So other can contribute easily

@nihaltp
Copy link
Contributor Author

nihaltp commented Oct 12, 2025

can you merge this @anshumanjadiya1102

@anshumanjadiya1102
Copy link
Member

OK @nihaltp

@anshumanjadiya1102 anshumanjadiya1102 merged commit 8e7a438 into Drive-for-Java:main Oct 12, 2025
@nihaltp nihaltp deleted the feat/title branch October 12, 2025 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest This is for Hacktoberfest hacktoberfest-accepted This is for Hacktoberfest

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants