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.

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

FIX: use the new names for the scripts #23

Merged
merged 2 commits into from
Dec 14, 2022

Conversation

amtoine
Copy link
Owner

@amtoine amtoine commented Nov 20, 2022

This PR changes the names of the calls to the GOAT scripts as discussed in amtoine/scripts#33.

I let this PR a draft for the time being and will request a review from you @atxr as soon as

is closed 😋

the script to fix the names (8b9bf4f)

open map.csv
| each { |it|
  [$it.old $it.new] | str collect " -> "
  git ls-files
  | lines
  | each { |file|
    sd -s -f w $it.old $it.new $file
  }
}

the map

raw file: map.csv

old new
amtoine-apply-to-hunks goat-apply-to-hunks
amtoine-battery gbat
amtoine-compute-color-palette goat-compute-color-palette
amtoine-countdown goat-countdown
amtoine-dmrun gmrun
amtoine-gh-notify gghn
amtoine-hdmi grandr
amtoine-lcfg goat-lcfg
amtoine-lfrun glf
amtoine-list-monitors glm
amtoine-macho gmacho
amtoine-mocp gmocp
amtoine-open gopen
amtoine-pcm gpcm
amtoine-pdf gpdf
amtoine-pedit gedit
amtoine-prompt gask
amtoine-scrot gshot
amtoine-slock glock
amtoine-sound gmixer
amtoine-spark gspark
amtoine-surf-open gsurf
amtoine-sys-stats gsstats
amtoine-syssync gssync
amtoine-tcfg goat-tcfg
amtoine-time-date gdate
amtoine-togkb gtogkb
amtoine-tr2md gtr2md
amtoine-update gsupdate
amtoine-upl gpkg
amtoine-vdl gvdl
amtoine-weather-get gweather
amtoine-wfzf gwfzf
amtoine-wtldr gwtldr
amtoine-wvenv gwvenv
amtoine-xal gxal
amtoine-xtcl gxtcl
amtoine-ytdl gytdl

@amtoine
Copy link
Owner Author

amtoine commented Nov 27, 2022

in cc89c3a, i've moved the config files of some GOAT scripts to their new locations, defined in amtoine/scripts#38.

@amtoine amtoine marked this pull request as ready for review December 7, 2022 17:52
@amtoine amtoine requested a review from atxr December 7, 2022 17:52
@amtoine
Copy link
Owner Author

amtoine commented Dec 7, 2022

@atxr
as amtoine/scripts#38 has been closed, we need to implement this one 😉

@amtoine amtoine added bug Something isn't working and removed needs-triage An issue or PR without label labels Dec 9, 2022
@amtoine amtoine merged commit 259fc5a into main Dec 14, 2022
@amtoine amtoine deleted the fix/change-the-names-of-the-scripts branch December 14, 2022 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant