Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 15, 2025

Shortens the port flag from -port to -p and defaults to port 3844 instead of hardcoded 8080. Includes merge with main branch to incorporate project rename and latest features.

Changes:

  • Flag renamed: flag.String("p", "3844", ...)
  • README updated to reflect new flag and port 3844 default
  • Binary name updated from diffs-cli to diff-server (from main branch merge)
  • Merged latest changes from main including error handling improvements and Diff2Html rendering

Example:

# Before
./diffs-cli -port 9000

# After  
./diff-server -p 9000

# Default port 3844
./diff-server
# Starting server on http://localhost:3844

The -p flag allows for easy port customization while maintaining a consistent default port.

Original prompt

Change the port param to just "-p". Default to a random number.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: alexec <1142830+alexec@users.noreply.github.com>
Copilot AI changed the title [WIP] Change port parameter to use '-p' with random default Change port flag to -p with random ephemeral port default Nov 15, 2025
Copilot AI requested a review from alexec November 15, 2025 20:13
@alexec
Copy link
Contributor

alexec commented Nov 15, 2025

@copilot Make the port number 3844

@alexec
Copy link
Contributor

alexec commented Nov 15, 2025

@copilot please fetch main and resolve the conflicts.

Co-authored-by: alexec <1142830+alexec@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Nov 15, 2025

@copilot Make the port number 3844

Changed the default port to 3844 in ce9f3e8.

The server now starts on port 3844 by default:

$ ./diffs-cli -h
Usage of ./diffs-cli:
  -C string
    	Directory to scan for git repositories (default ".")
  -p string
    	Port to listen on (default "3844")

Copilot AI changed the title Change port flag to -p with random ephemeral port default Change port flag to -p with default port 3844 Nov 15, 2025
@alexec alexec marked this pull request as ready for review November 15, 2025 20:18
@alexec alexec enabled auto-merge (squash) November 15, 2025 20:19
Co-authored-by: alexec <1142830+alexec@users.noreply.github.com>
auto-merge was automatically disabled November 15, 2025 20:21

Head branch was pushed to by a user without write access

Copy link
Contributor Author

Copilot AI commented Nov 15, 2025

@copilot please fetch main and resolve the conflicts.

Merged main branch and resolved conflicts in d23c8dd.

The merge incorporated:

  • Project rename from diffs-cli to diff-server
  • Error handling improvements
  • Diff2Html rendering features
  • All other changes from main

The port flag -p with default 3844 is preserved. All tests pass.

$ ./diff-server -h
Usage of ./diff-server:
  -C string
    	Directory to scan for git repositories (default ".")
  -p string
    	Port to listen on (default "3844")

Copilot AI requested a review from alexec November 15, 2025 20:24
@alexec alexec merged commit afc8966 into main Nov 15, 2025
1 check passed
@alexec alexec deleted the copilot/change-port-param-to-p branch November 15, 2025 20:26
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