Skip to content

feat(landprovider): Adding land provider extension interface#104

Closed
kevinlnew wants to merge 4 commits into
mainfrom
kevin.new/landprovider
Closed

feat(landprovider): Adding land provider extension interface#104
kevinlnew wants to merge 4 commits into
mainfrom
kevin.new/landprovider

Conversation

@kevinlnew
Copy link
Copy Markdown
Contributor

@kevinlnew kevinlnew commented Feb 27, 2026

Summary

feat(landprovider): Adding land provider extension interface

Why?
LandProvider is the extension for landing changes.

What?

  • Added the LandProvider interface with a basic Land method that takes []LandEntry
  • Added LandEntry, a basic tuple construction for landing strategy and the change being landed
  • Added a land-specific error type ErrLandRejected, this is a specific type that will affect the control flow of the merge controller.

Test Plan

Unit tests, CI.

Issues

Stack

  1. @ feat(landprovider): Adding land provider extension interface #104
  2. feat(landprovider): basic github implementation #109
  3. feat(controller): hooking up merge controller #105

@kevinlnew kevinlnew force-pushed the kevin.new/landprovider branch 2 times, most recently from e8b31bc to bfbfc86 Compare February 27, 2026 21:57
Comment thread entity/land_entry.go
@kevinlnew kevinlnew marked this pull request as ready for review March 2, 2026 18:46
@kevinlnew kevinlnew requested review from a team and behinddwalls as code owners March 2, 2026 18:46
Comment thread extension/landprovider/land_provider.go
@kevinlnew kevinlnew requested a review from behinddwalls March 2, 2026 20:41
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