Open
Conversation
|
|
||
| ```bash | ||
| scribbly watch --config custom.scribblyrc.json | ||
| scribbly watch --config customscribbly.config.json |
There was a problem hiding this comment.
The watch example config filename looks like it lost a separator during the rename. customscribbly.config.json is likely a typo and should probably be custom.scribbly.config.json (or another clearly separated name), otherwise users copying this command will pass a non-existent file.
Suggested change
| scribbly watch --config customscribbly.config.json | |
| scribbly watch --config custom.scribbly.config.json |
Prompt To Fix With AI
This is a comment left during a code review.
Path: README.md
Line: 154:154
Comment:
The watch example config filename looks like it lost a separator during the rename. `customscribbly.config.json` is likely a typo and should probably be `custom.scribbly.config.json` (or another clearly separated name), otherwise users copying this command will pass a non-existent file.
```suggestion
scribbly watch --config custom.scribbly.config.json
```
How can I resolve this? If you propose a fix, please make it concise.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changed
.scribblyrc.json→scribbly.config.jsonin README.Scout-detected fix.
Greptile Overview
Greptile Summary
Updates README to document the correct config filename (
scribbly.config.json) instead of the old.scribblyrc.json.Findings
README.md:154The watch-mode example usescustomscribbly.config.json(missing separator). Users copy/pasting will likely reference a non-existent file; should be something likecustom.scribbly.config.json.Confidence Score: 4/5
customscribbly.config.json) will mislead users and should be corrected before merge.Important Files Changed
.scribblyrc.jsontoscribbly.config.json; introduces a typo in the watch example (customscribbly.config.json).Sequence Diagram