Skip to content
This repository was archived by the owner on Jan 8, 2026. It is now read-only.

Remove Palantir stacktrace dependency and use standard Go error handling#9

Merged
wboayue merged 2 commits intomainfrom
update-error-handling
Aug 27, 2025
Merged

Remove Palantir stacktrace dependency and use standard Go error handling#9
wboayue merged 2 commits intomainfrom
update-error-handling

Conversation

@wboayue
Copy link
Owner

@wboayue wboayue commented Aug 27, 2025

Summary

  • Replaced Palantir stacktrace with standard Go error wrapping using fmt.Errorf with %w verb
  • Updated dependencies (removed unused Palantir dependency, updated test dependencies)
  • Added CLAUDE.md documentation file for AI assistant guidance

Changes

  • Error handling refactor: Migrated all error handling in client.go and transport.go from stacktrace.Propagate/NewError to idiomatic Go 1.13+ error wrapping
  • Dependency cleanup: Removed github.com/palantir/stacktrace dependency and updated stretchr/testify to v1.11.0
  • Documentation: Added comprehensive CLAUDE.md file with project overview, architecture, and development guidelines

@wboayue wboayue merged commit d1afc0f into main Aug 27, 2025
3 checks passed
@wboayue wboayue deleted the update-error-handling branch August 27, 2025 04:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant