Skip to content

feat: add Dagger lint and test pipelines#9

Closed
MChorfa wants to merge 0 commit intoanthropics:release-please--branches--main--changes--nextfrom
MChorfa:feat/dagger-lint-test-pipeline
Closed

feat: add Dagger lint and test pipelines#9
MChorfa wants to merge 0 commit intoanthropics:release-please--branches--main--changes--nextfrom
MChorfa:feat/dagger-lint-test-pipeline

Conversation

@MChorfa
Copy link
Copy Markdown

@MChorfa MChorfa commented Apr 1, 2026

Summary

Adds comprehensive lint and test pipelines using Dagger.

Changes

  • dagger/pipelines/lint.go: Go build linting
  • dagger/pipelines/test.go: Full test suite with mock server
  • dagger/pipelines/test.go: Fast unit tests (TestFast)
  • dagger/main.go: Refactored to use pipelines package

Features

  • Lint: Runs go build ./...
  • Test: Full tests with mock server service
  • TestFast: Short unit tests only

Dependencies

PR 7 (Dagger Bootstrap)

Risk

Medium - builds on Dagger infrastructure.

@stainless-app stainless-app Bot force-pushed the release-please--branches--main--changes--next branch 16 times, most recently from 9271554 to 783bf0a Compare April 9, 2026 00:27
@stainless-app stainless-app Bot requested a review from a team as a April 9, 2026 00:27
@MChorfa
Copy link
Copy Markdown
Author

MChorfa commented Apr 9, 2026

Closing this PR due to merge conflicts with upstream CI/CD workflow files.

The Dagger module conflicts with:

  • .github/workflows/ci.yml (upstream has their own CI)
  • dagger/ directory additions may overlap with upstream changes

Since upstream uses Stainless for SDK generation and has established CI, adding Dagger would require coordination with their workflow. The remaining rebased PRs (#5, #6, #7) are smaller focused improvements that don't conflict with CI infrastructure.

@MChorfa MChorfa closed this Apr 9, 2026
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.

1 participant