Coordinate-Cat / dotfiles-devday

でぶでい用教材
1 stars 1 forks source link

imo: Wrong Readme #4

Closed yasunori0418 closed 6 months ago

yasunori0418 commented 6 months ago

@Coordinate-Cat git cloneがコメントになってしまっている…。

-# git clone
-git@github.com:Coordinate-Cat/dotfiles-devday.git
+git clone git@github.com:Coordinate-Cat/dotfiles-devday.git

https://github.com/Coordinate-Cat/dotfiles-devday/blob/main/README.md?plain=1#L29

ln -s /$HOME/dotfiles/.zshrc /$HOME/.zshrc

https://github.com/Coordinate-Cat/dotfiles-devday/blob/main/README.md?plain=1#L55

ln -s /$HOME/dotfiles-devday/powerlevel10k /$HOME/powerlevel10k

これらに共通する/$HOMEに含まれる/は不要だと思う。 これがシェルとして展開されると、//home/userのようなパスになるので綺麗ではない。


あと、ここのsymlinkは不要かも。 https://github.com/Coordinate-Cat/dotfiles-devday/blob/main/README.md?plain=1#L55

ln -s /$HOME/dotfiles-devday/powerlevel10k /$HOME/powerlevel10k

powerlevel10kはinstall.sh~/.zsh/plugins/github.com/romkatv/powerlevel10kに配置されるから、このsymlinkの説明があると混乱させてしまうかも。 .zshrcでも配置予定のパスをベースにp10kを読み込むようにもなっている。

yasunori0418 commented 6 months ago

https://github.com/Coordinate-Cat/dotfiles-devday/blob/main/README.md?plain=1#L15-L16

実は、install.shには実行権限を付けてしまっているので、次のように実行できる。

 # install.sh を実行
-zsh ./install.sh
+./install.sh

#!/bin/bashでshebangが明記されているので、install.shの文法にはbash由来の物を使っている。 実行時の推奨は./install.shが良い。