linutil icon indicating copy to clipboard operation
linutil copied to clipboard

Published 20 hours ago • verified publisher icon ChrisTitusTech

Remove every bashism

Open nnyyxxxx opened this issue 5 months ago • 6 comments

Type of Change

  • [x] New feature
  • [x] Bug fix
  • [x] Refactoring
  • [x] Hotfix

Description

I replaced every bashism with posix alternatives, I removed the custom color codes that were in almost every file (utilities), I refactored a lot of stuff (utilities), and I made it rely on the common script color codes instead of the custom ones (utilities).

Testing

Tested on bash, zsh, and dash.

Rechecked every file using checkbashisms package

Impact

Greatly impacts the changed sections.

Issue related to PR

  • Resolves #246
  • Resolves #147
  • Resolves #312
  • Resolves #232
  • Resolves #383
  • Resolves #350
  • Closes #278

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 06 '24 06:09 nnyyxxxx