Skip to content

Remove a bit from pre-commit that husky says we should#4286

Merged
jamadeo merged 1 commit into
mainfrom
jackamadeo/husky-update
Aug 25, 2025
Merged

Remove a bit from pre-commit that husky says we should#4286
jamadeo merged 1 commit into
mainfrom
jackamadeo/husky-update

Conversation

@jamadeo

@jamadeo jamadeo commented Aug 22, 2025

Copy link
Copy Markdown
Member

When I made a commit, the husky pre-commit hook warned me to delete these lines.

@zanesq zanesq 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.

Thanks! Hopefully the hook still works after this change?

@jamadeo

jamadeo commented Aug 25, 2025

Copy link
Copy Markdown
Member Author

Thanks! Hopefully the hook still works after this change?

Yep, tested by introducing type/lint errors and running git commit from both ui/desktop and repo root.

@jamadeo jamadeo merged commit 4ef3746 into main Aug 25, 2025
11 checks passed
@jamadeo jamadeo deleted the jackamadeo/husky-update branch August 25, 2025 12:36
zanesq added a commit that referenced this pull request Aug 25, 2025
* 'main' of github.com:block/goose:
  fix slow typing with long sessions (#4291)
  docs: improvements to release docs (#4317)
  blog: mcp-ui recap blog (#4146)
  added more logging for recipe creation failures (#4299)
  Remove a bit from pre-commit that husky says we should (#4286)
  feat: autovisualiser of structured data with mcp-ui (#4153)
  docs: Plan tutorial (#4309)
  Extensions Modal Improvements (#4293)
  docs: fixed cicd tutorial pipeline in docs (#4223)
  Read oltp config from config and env (#4292)

# Conflicts:
#	ui/desktop/src/App.tsx
shellz-n-stuff pushed a commit to shellz-n-stuff/goose that referenced this pull request Aug 27, 2025
Signed-off-by: Alex Rosenzweig <arosenzweig@squareup.com>
dorien-koelemeijer pushed a commit to dorien-koelemeijer/goose that referenced this pull request Sep 2, 2025
Signed-off-by: Dorien Koelemeijer <dkoelemeijer@squareup.com>
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.

2 participants