Skip to content

Remove conformance tests - #331

Merged
williammartin merged 1 commit into
mainfrom
wm/remove-conformance-tests
Apr 22, 2025
Merged

Remove conformance tests#331
williammartin merged 1 commit into
mainfrom
wm/remove-conformance-tests

Conversation

@williammartin

Copy link
Copy Markdown
Collaborator

Description

The conformance tests were introduced in #48 in order to ensure that when we released this project it was offering the same / very close to the same functionality as the original anthropic project. This was a point in time issue, and we no longer need these tests. In fact they are likely to cause confusion going forward.

#330 offers a slightly different and more comprehensive approach to gain confidence on some overlapping areas (blackbox running and interacting).

CopilotAI review requested due to automatic review settings April 22, 2025 15:52
@williammartin
williammartin requested a review from a team as a code ownerApril 22, 2025 15:52

CopilotAI 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.

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

Files not reviewed (1)
  • go.mod: Language not supported

@williammartin
williammartin merged commit 71b0075 into mainApr 22, 2025
@williammartin
williammartin deleted the wm/remove-conformance-tests branch April 22, 2025 16:01
DaleSeo pushed a commit to DaleSeo/github-mcp-server that referenced this pull request Oct 24, 2025
test: enable unit tests and make validate tests flexible in terms of working directory
Sign up for freeto 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

@williammartin@toby