linutil icon indicating copy to clipboard operation
linutil copied to clipboard

Use -e to keep synergy with every other script

Open nnyyxxxx opened this issue 1 year ago • 6 comments

Type of Change

  • [x] New feature
  • [x] Refactoring
  • [x] Hotfix
  • [x] Security patch

Description

No explanation needed. Ref - PR Title

  • Resolves #407

Checklist

  • [x] My code adheres to the coding and style guidelines of the project.
  • [x] I have performed a self-review of my own code.
  • [x] My changes generate no errors/warnings/merge conflicts.

nnyyxxxx avatar Sep 16 '24 01:09 nnyyxxxx

Duplicate of #388

Not a duplicate, that was committed after this PR, I suggested changes as well. @jeevithakannan2 You should keep your pull requests focused. https://github.com/ChrisTitusTech/linutil/pull/388/commits/cfd6cca853a484e1d4fba0e87ca46a74d070e3f5

ghost avatar Sep 16 '24 02:09 ghost

No need of two separate PRs for a simple change

jeevithakannan2 avatar Sep 16 '24 02:09 jeevithakannan2

Reverted as per your wish

jeevithakannan2 avatar Sep 16 '24 03:09 jeevithakannan2

@nnyyxxxx You are wasting time and it's a simple fix, it doesn't need two separate PRs. I am not stealing your commits and I will not. I don't know why you make small things a big issue and accuse people. I am not going to argue with you over this small issue not worth my time

jeevithakannan2 avatar Sep 16 '24 03:09 jeevithakannan2

Re-opened due to revert.

ghost avatar Sep 16 '24 04:09 ghost

@nnyyxxxx You are wasting time and it's a simple fix, it doesn't need two separate PRs. I am not stealing your commits and I will not. I don't know why you make small things a big issue and accuse people. I am not going to argue with you over this small issue not worth my time

Never said you were, I'm only saying that this is not a duplicate, that commit that I linked was made an hour after this PR was opened with the fix.

ghost avatar Sep 16 '24 04:09 ghost