Skip to content

Update#102

Open
amina69 wants to merge 1 commit into
Miracle656:mainfrom
amina69:main
Open

Update#102
amina69 wants to merge 1 commit into
Miracle656:mainfrom
amina69:main

Conversation

@amina69

@amina69 amina69 commented Jun 2, 2026

Copy link
Copy Markdown

close #77
Dredd run is green
Drift flagged on PR

@amina69 amina69 closed this Jun 3, 2026
@amina69 amina69 reopened this Jun 3, 2026

@Miracle656 Miracle656 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This PR can't close #77. #77 is "API contract tests with Dredd", which needs a Dredd configuration and a contract-test suite wired into CI — but the only change here is to README.md, and it actually breaks the setup instruction: cp .env.example .env becomes cp .env.example.env (the space is removed, so it copies to a single nonsensically-named file). CI is also failing (typecheck + integration).

To address #77, this needs the actual Dredd setup (e.g. a dredd.yml pointing at the OpenAPI spec + a CI job running dredd), not a README edit. Please either replace this with the real implementation or close it. Also revert the .env command change — the original cp .env.example .env was correct.

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.

API contract tests with Dredd

2 participants