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

fix: removed the panel in the output so that the user won't have unnecessary pane borders in the copied content #109

Merged
merged 2 commits into from
Oct 3, 2024

Conversation

lifeizhou-ap
Copy link
Collaborator

screenshots after removal
shell
Screenshot 2024-10-03 at 5 45 20 PM

patch_file
Screenshot 2024-10-03 at 5 46 13 PM

write_file
Screenshot 2024-10-03 at 5 46 47 PM

Copy link
Collaborator

@michaelneale michaelneale left a comment

Choose a reason for hiding this comment

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

I like this - haven’t had a chance to try it myself but seems like a great idea

@baxen baxen force-pushed the lifei/remove-pane-output-for-copy-content branch from 8429792 to 4f436ed Compare October 3, 2024 18:27
Copy link
Collaborator

@baxen baxen left a comment

Choose a reason for hiding this comment

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

Looks great!

I added a small style tweak on the rules

@lifeizhou-ap lifeizhou-ap merged commit 9da4881 into main Oct 3, 2024
4 checks passed
lifeizhou-ap added a commit that referenced this pull request Oct 4, 2024
* main:
  fix: exit the goose and show the error message when provider environment variable is not set (#103)
  fix: Update OpenAI pricing per https://openai.com/api/pricing/ (#110)
  fix: update developer tool prompts to use plan task status to match allowable statuses update_plan tool call (#107)
  fix: removed the panel in the output so that the user won't have unnecessary pane borders in the copied content (#109)
  docs: update links to exchange to the new location (#108)
  chore: setup workspace for exchange (#105)
@lifeizhou-ap lifeizhou-ap deleted the lifei/remove-pane-output-for-copy-content branch October 7, 2024 03:13
lily-de pushed a commit that referenced this pull request Oct 7, 2024
…cessary pane borders in the copied content (#109)

Co-authored-by: Bradley Axen <baxen@squareup.com>
lukealvoeiro added a commit that referenced this pull request Oct 9, 2024
* main: (41 commits)
  chore: Add goose providers list command (#116)
  docs: working ollama for desktop (#125)
  docs: format and clean up warnings/errors (#120)
  docs: update deploy workflow (#124)
  feat: Implement a goose run command (#121)
  feat: saved api_key to keychain for user (#104)
  docs: add callout plugin (#119)
  chore: add a page to docs for Goose application examples (#117)
  fix: exit the goose and show the error message when provider environment variable is not set (#103)
  fix: Update OpenAI pricing per https://openai.com/api/pricing/ (#110)
  fix: update developer tool prompts to use plan task status to match allowable statuses update_plan tool call (#107)
  fix: removed the panel in the output so that the user won't have unnecessary pane borders in the copied content (#109)
  docs: update links to exchange to the new location (#108)
  chore: setup workspace for exchange (#105)
  fix: resolve uvx when using a git client or IDE (#98)
  ci: add include-markdown for mkdocs (#100)
  chore: fix broken badge on readme (#102)
  feat: add global optional user goosehints file (#73)
  docs: update docs (#99)
  chore(release): release 0.9.3 (#97)
  ...
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.

3 participants