Skip to content

Add the ability to run update-readme command on a different project - #82

Merged
johnpryan merged 3 commits into
mainfrom
update-update-readme
Apr 22, 2026
Merged

johnpryan merged 3 commits into
mainfrom
update-update-readme

Conversation

@johnpryan

Copy link
Copy Markdown
Collaborator

This is needed to update the dart-lang/skills README: dart-lang/skills#6

@johnpryan
johnpryan requested a review from jwren April 14, 2026 18:13

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request adds a --readme command-line option to the UpdateReadmeCommand and enhances the logic for locating the README.md file by checking the output directory and its parent. A review comment identifies a compilation error caused by a missing import for the path package and suggests a more idiomatic way to handle the file search logic using firstWhere.

Comment thread tool/generator/lib/src/commands/update_readme_command.dart
@johnpryan
johnpryan merged commit 8a9191f into main Apr 22, 2026
6 checks passed
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