Skip to content

Conversation

@varonix0
Copy link
Member

Description 📣

Renamed service-log-file to log-file to be more compliant with CLI standards.

Type ✨

  • Bug fix
  • New feature
  • Improvement
  • Breaking change
  • Documentation

@varonix0 varonix0 self-assigned this Nov 28, 2025
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Nov 28, 2025

Greptile Overview

Greptile Summary

This PR renames the --service-log-file flag to --log-file for better CLI standards compliance. The change is consistently applied across gateway and relay systemd installation commands, along with their error messages.

  • Renamed flag from service-log-file to log-file in gateway systemd install and relay systemd install commands
  • Updated all error messages and help text to reference the new flag name
  • Updated validation error message in systemd.go utility

The changes are straightforward, consistent, and maintain backward compatibility since this is a flag rename (users will simply need to use the new flag name).

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • The changes are purely cosmetic renaming of a CLI flag from service-log-file to log-file. All references have been consistently updated across flag definitions, error messages, and validation logic. No functional logic was changed, and the refactoring improves CLI standards compliance.
  • No files require special attention

Important Files Changed

File Analysis

Filename Score Overview
packages/cmd/gateway.go 5/5 Renamed flag from service-log-file to log-file in gateway systemd install command
packages/cmd/relay.go 5/5 Renamed flag from service-log-file to log-file in relay systemd install command
packages/util/systemd.go 5/5 Updated error message to reference log-file instead of service-log-file

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

3 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@varonix0 varonix0 requested a review from maidul98 November 28, 2025 00:02
@varonix0 varonix0 merged commit f84b4a0 into main Nov 28, 2025
4 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.

3 participants