webinstall / webi-installers

Primary and community-submitted packages for webinstall.dev
https://webinstall.dev
Mozilla Public License 2.0
1.97k stars 211 forks source link

Simple refactor: find-and-replace `whoami` with `id -u -n` in scripts #854

Open coolaj86 opened 5 months ago

coolaj86 commented 5 months ago

whoami is generally available on all Unix-y systems, but id -u -n is the POSIX standard with guaranteed behavior.

This is literally as simple as doing a find-and-replace across the repo (such as with sd, or any code editor tool).

Finding the files with whoami:

git clone https://github.com/webinstall/webi-installers ./webi-installers
pushd webi-installers/
rg 'whoami' || grep -r 'whoami' .

For reference: https://pubs.opengroup.org/onlinepubs/009695399/utilities/id.html

shuchitpant commented 1 day ago

Hi @coolaj86,

I would like to work on this issue.

Could you please assign this issue to me?

shuchitpant commented 1 day ago

Hi AJ,

I've raised a PR (#915 ), that addresses this issue, kindly go through this and let me know if there are any changes.