Skip to content

Hide the custom fetcher for now#1175

Merged
frederikprijck merged 5 commits into
masterfrom
chore/hide-fetcher
Sep 11, 2025
Merged

Hide the custom fetcher for now#1175
frederikprijck merged 5 commits into
masterfrom
chore/hide-fetcher

Conversation

@frederikprijck
Copy link
Copy Markdown
Member

Changes

As we want to change the public contract of the fetcher, we are hiding it for now until this is resolved.

References

Please include relevant links supporting this change such as a:

  • support ticket
  • community post
  • StackOverflow post
  • support forum thread

Testing

Please describe how this can be tested by reviewers. Be specific about anything not tested and reasons why. If this library has unit and/or integration testing, tests should be added for new functionality and existing tests should complete without errors.

  • This change adds unit test coverage
  • This change adds integration test coverage

Checklist

@frederikprijck frederikprijck requested a review from a team as a code owner September 8, 2025 15:56
developerkunal
developerkunal previously approved these changes Sep 8, 2025
Copy link
Copy Markdown
Contributor

@developerkunal developerkunal left a comment

Choose a reason for hiding this comment

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

LGTM

@frederikprijck frederikprijck marked this pull request as draft September 8, 2025 16:07
@frederikprijck frederikprijck marked this pull request as ready for review September 8, 2025 16:12
Copy link
Copy Markdown
Contributor

@developerkunal developerkunal left a comment

Choose a reason for hiding this comment

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

LGTM

@frederikprijck frederikprijck enabled auto-merge (squash) September 11, 2025 12:47
@frederikprijck frederikprijck merged commit 4540ea7 into master Sep 11, 2025
11 of 12 checks passed
@frederikprijck frederikprijck deleted the chore/hide-fetcher branch September 11, 2025 12:48
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