Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

tac: don't print version number in usage string #981

Merged
merged 2 commits into from
Mar 5, 2025
Merged

tac: don't print version number in usage string #981

merged 2 commits into from
Mar 5, 2025

Conversation

mknos
Copy link
Contributor

@mknos mknos commented Mar 5, 2025

  • If I type a bad option I want to see the correct options, not the program version
  • As done in other scripts using getopts(), support --version option by adding a VERSION_MESSAGE() function

@mknos
tac: don't print version number in usage string
a171bb1 
* If I type a bad option I want to see the correct options, not the program version
* As done in other scripts using getopts(), support --version option by adding a VERSION_MESSAGE() function
@mknos mknos temporarily deployed to housekeeping March 5, 2025 02:55 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 02:55 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 02:55 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 02:55 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 02:55 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 02:55 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 02:55 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 02:55 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 02:55 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 02:55 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 02:55 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 02:55 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 02:55 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 02:55 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 02:55 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 02:55 — with GitHub Actions Inactive
@mknos mknos had a problem deploying to automated_testing March 5, 2025 02:55 — with GitHub Actions Error
@mknos mknos temporarily deployed to automated_testing March 5, 2025 02:55 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 02:55 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 02:55 — with GitHub Actions Inactive
@mknos mknos had a problem deploying to automated_testing March 5, 2025 02:55 — with GitHub Actions Failure
@github-actions github-actions bot added Type: enhancement improve a feature that already exists Priority: low get to this whenever Program: tac The tac program Status: needs verification issue needs to be verified Type: bug an existing feature does not work labels Mar 5, 2025
@github-actions github-actions bot requested a review from briandfoy March 5, 2025 02:56
@coveralls
Copy link

coveralls commented Mar 5, 2025 •
edited
Loading

Pull Request Test Coverage Report for Build 13667573029

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 70.951%
Totals Coverage Status
Change from base Build 13656102414: 0.0%
Covered Lines: 403
Relevant Lines: 568
💛 - Coveralls

@coveralls
Copy link

Pull Request Test Coverage Report for Build 13667488487

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.9%) to 70.088%
Totals Coverage Status
Change from base Build 13656102414: -0.9%
Covered Lines: 396
Relevant Lines: 565
💛 - Coveralls

@mknos mknos temporarily deployed to automated_testing March 5, 2025 03:03 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 03:03 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 03:03 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 03:03 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 03:03 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 03:03 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 03:03 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 03:03 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 03:03 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 03:03 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 03:03 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 03:03 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 03:03 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 03:03 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 03:03 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 03:03 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 03:03 — with GitHub Actions Inactive
@mknos mknos temporarily deployed to automated_testing March 5, 2025 03:03 — with GitHub Actions Inactive
@mknos mknos had a problem deploying to automated_testing March 5, 2025 03:03 — with GitHub Actions Failure
@mknos mknos had a problem deploying to automated_testing March 5, 2025 03:03 — with GitHub Actions Error
@briandfoy briandfoy self-assigned this Mar 5, 2025
@briandfoy
Copy link
Owner

briandfoy commented Mar 5, 2025 •
edited
Loading

change: don't print version number in usage string; add --version option

@briandfoy briandfoy merged commit 991d131 into briandfoy:master Mar 5, 2025
20 of 22 checks passed
@github-actions github-actions bot added Status: accepted The fix is accepted and removed Status: needs verification issue needs to be verified Priority: low get to this whenever labels Mar 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@briandfoy briandfoy briandfoy approved these changes

Assignees

@briandfoy briandfoy

Labels
Program: tac The tac program Status: accepted The fix is accepted Type: bug an existing feature does not work Type: enhancement improve a feature that already exists
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mknos @coveralls @briandfoy