Skip to content

feat(schema): Extract PatternMatchError typed error from FilterDfs#2458

Merged
kodiakhq[bot] merged 1 commit intomainfrom
fix/add-extractable-patternmatcherror
Mar 26, 2026
Merged

feat(schema): Extract PatternMatchError typed error from FilterDfs#2458
kodiakhq[bot] merged 1 commit intomainfrom
fix/add-extractable-patternmatcherror

Conversation

@disq
Copy link
Member

@disq disq commented Mar 26, 2026

Replace plain fmt.Errorf calls with a structured PatternMatchError type so callers can use errors.As/errors.Is to extract/validate the invalid pattern and which config key (tables or skip_tables) caused the mismatch.

Replace plain fmt.Errorf calls with a structured PatternMatchError type
so callers can use errors.As/errors.Is to extract the invalid pattern
and which config key (tables or skip_tables) caused the mismatch.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@disq disq requested a review from a team as a code owner March 26, 2026 09:48
@disq disq requested review from erezrokah and murarustefaan March 26, 2026 09:48
Copy link
Member

@murarustefaan murarustefaan left a comment

Choose a reason for hiding this comment

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

Nice touch, actually.

@disq disq added the automerge label Mar 26, 2026
@kodiakhq kodiakhq bot merged commit 8b3e522 into main Mar 26, 2026
9 checks passed
@kodiakhq kodiakhq bot deleted the fix/add-extractable-patternmatcherror branch March 26, 2026 10:06
disq pushed a commit that referenced this pull request Mar 26, 2026
🤖 I have created a release *beep* *boop*
---


##
[4.95.0](v4.94.7...v4.95.0)
(2026-03-26)


### Features

* **schema:** Extract `PatternMatchError` typed error from `FilterDfs`
([#2458](#2458))
([8b3e522](8b3e522))


### Bug Fixes

* **deps:** Update module github.com/cloudquery/codegen to v0.4.0
([#2455](#2455))
([041d331](041d331))
* **deps:** Update module github.com/cloudquery/plugin-pb-go to v1.27.11
([#2453](#2453))
([1f94119](1f94119))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants