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

Add ability to ignore patterns directly from cli #4

Merged
merged 4 commits into from
Apr 8, 2024

Conversation

dipam7
Copy link
Contributor

@dipam7 dipam7 commented Mar 23, 2024

Solves #1

@simonw simonw merged commit f8af0fa into simonw:main Apr 8, 2024
5 checks passed
@simonw
Copy link
Owner

simonw commented Apr 8, 2024

I applied your changes by downloading the .diff for this PR:

wget 'https://patch-diff.githubusercontent.com/raw/simonw/files-to-prompt/pull/4.diff'

And then prompting it to apply those changes to the latest main like this:

files-to-prompt 4.diff files_to_prompt/cli.py tests/test_files_to_prompt.py | \
  llm -m opus --system 'Apply the change described in the diff to the project - return updated cli.py and tests'

That didn't get it right so I ran this:

llm -c 'you undid the change where it could handle multiple paths - I want to keep that, I only want to add the new --ignore-patterns option'

Transcript: https://gist.github.com/simonw/8b8394e320c895c792736e7e85c40c23

@simonw
Copy link
Owner

simonw commented Apr 8, 2024

I'm going to rename the option to --ignore, refs #1

simonw added a commit that referenced this pull request Apr 8, 2024
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.

None yet

2 participants