Skip to content

docs: update README #23

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

Merged
merged 3 commits into from
Jun 6, 2025
Merged

docs: update README #23

merged 3 commits into from
Jun 6, 2025

Conversation

guitarrapc
Copy link
Owner

No description provided.

@Copilot Copilot AI review requested due to automatic review settings June 6, 2025 12:16
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR enhances the README (English and Japanese) by documenting the new dry-run mode, adding the corresponding flag to the options table, and expanding the default ignore list.

  • Added a dry-run usage example in both README.md and README_ja.md
  • Introduced the --dry-run, -d flag in the options table
  • Extended the ignore list with OS-specific, temporary, and VCS folders

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
README.md Added dry-run example snippet, option table entry, and expanded ignore list
README_ja.md 同様のdry-run例、オプション表エントリ、および除外リストの拡張を追加
Comments suppressed due to low confidence (2)

README.md:165

  • The example path includes a literal HOME directory segment in the source path which is likely unintended. It should reference the dotfiles folder directly, e.g., /home/user/dotfiles/.config/git/config.
[o] [DRY-RUN] Would create file symlink: /home/user/.config/git/config -> /home/user/dotfiles/HOME/.config/git/config

README_ja.md:165

  • 例示パスに誤って HOME ディレクトリが含まれています。正しくは /home/user/dotfiles/.config/git/config のように、dotfilesディレクトリ直下を指すべきです。
[o] [DRY-RUN] Would create file symlink: /home/user/.config/git/config -> /home/user/dotfiles/HOME/.config/git/config

README.md Outdated
@@ -239,6 +253,9 @@ LICENSE
The following files and directories are automatically excluded:
- Directories starting with `.git` (like `.github`)
- Non-dotfiles in the root directory
- OS-specific files like `.DS_Store` (macOS) and `Thumbs.db` (Windows)
- Temporary files like `*.bak`, `*.tmp`, and vim swap files
- Version control system folders (`.git`, `.svn`, `.hg`)
Copy link
Preview

Copilot AI Jun 6, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nitpick] The ignore list duplicates .git which is already covered by the first bullet (directories starting with .git). Consider merging or removing one entry to avoid redundancy.

Copilot uses AI. Check for mistakes.

README_ja.md Outdated
Comment on lines 254 to 258
- `.git` で始まるディレクトリ(`.github` など)
- ルートディレクトリの非ドットファイル(先頭が `.` でないファイル)
- OS固有のファイル(macOSの`.DS_Store`、Windowsの`Thumbs.db`など)
- 一時ファイル(`*.bak`、`*.tmp`、vimのスワップファイルなど)
- バージョン管理システムのフォルダ(`.git`、`.svn`、`.hg`)
Copy link
Preview

Copilot AI Jun 6, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nitpick] 「.gitで始まるディレクトリ」との重複があります。冗長性を避けるため、項目を統合または一方を削除することを検討してください。

Suggested change
- `.git` で始まるディレクトリ(`.github` など)
- ルートディレクトリの非ドットファイル(先頭が `.` でないファイル)
- OS固有のファイル(macOSの`.DS_Store`、Windowsの`Thumbs.db`など)
- 一時ファイル(`*.bak``*.tmp`、vimのスワップファイルなど)
- バージョン管理システムのフォルダ(`.git``.svn``.hg`
- `.git` で始まるディレクトリ(例: `.github`)およびバージョン管理システムのフォルダ(例: `.git``.svn``.hg`
- ルートディレクトリの非ドットファイル(先頭が `.` でないファイル)
- OS固有のファイル(macOSの`.DS_Store`、Windowsの`Thumbs.db`など)
- 一時ファイル(`*.bak``*.tmp`、vimのスワップファイルなど)

Copilot uses AI. Check for mistakes.

@guitarrapc guitarrapc merged commit fd7ceea into main Jun 6, 2025
11 checks passed
@guitarrapc guitarrapc deleted the feature/update branch June 6, 2025 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant