Skip to content

feat(stovepipe): Record project validation facts - #735

Merged
prathshenoy merged 1 commit into
mainfrom
codem-476-record-project-results
Sep 24, 2026
Merged

prathshenoy merged 1 commit into
mainfrom
codem-476-record-project-results

Conversation

@prathshenoy

@prathshenoy prathshenoy commented Sep 21, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

What:

  • Record named project outcomes alongside final repository validation results.
  • Preserve repository-only validation for deployments without project attribution.

Why:

  • Enable consumers to query durable project failures when an integration supplies attribution.

Test Plan

  • Add unit tests.

Revert Plan

  • Revert this PR. This removes named project-result persistence while preserving repository validation.

Issues

@prathshenoy
prathshenoy marked this pull request as draft September 21, 2026 20:14
@mnoah1
mnoah1 added this pull request to stack #734 September 22, 2026 17:55
@mnoah1
mnoah1 force-pushed the codem-476-record-project-results branch from 6df17ce to 47b4e9f Compare September 22, 2026 19:06
Base automatically changed from prath.shenoy/codem-475-project-results to main September 22, 2026 19:21
@prathshenoy
prathshenoy force-pushed the codem-476-record-project-results branch 2 times, most recently from 73ab294 to 7b87743 Compare September 22, 2026 19:40
@prathshenoy
prathshenoy marked this pull request as ready for review September 22, 2026 19:41
Comment thread stovepipe/controller/record/record_test.go Outdated
Comment thread stovepipe/extension/projectresult/projectresult.go Outdated
Comment thread stovepipe/controller/record/record.go Outdated
Comment thread stovepipe/controller/record/record.go Outdated
@prathshenoy
prathshenoy force-pushed the codem-476-record-project-results branch 5 times, most recently from 1d06e12 to b5f9051 Compare September 23, 2026 20:09
@prathshenoy
prathshenoy added this pull request to the merge queue Sep 23, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Sep 23, 2026
## Summary

**What**:
- Record named project outcomes alongside final repository validation
results.
- Preserve repository-only validation for deployments without project
attribution.

**Why**:
- Enable consumers to query durable project failures when an integration
supplies attribution.

## Test Plan

- [x] Add unit tests.

## Revert Plan

- Revert this PR. This removes named project-result persistence while
preserving repository validation.

## Issues
-
[CODEM-529](https://linear.app/uber/issue/CODEM-529/persist-project-validation-results)
@prathshenoy
prathshenoy force-pushed the codem-476-record-project-results branch from b5f9051 to 3d3290b Compare September 24, 2026 20:33
@prathshenoy
prathshenoy added this pull request to the merge queue Sep 24, 2026
Merged via the queue into main with commit e14ccc9 Sep 24, 2026
16 checks passed
behinddwalls pushed a commit to behinddwalls/submitqueue that referenced this pull request Sep 25, 2026
## Summary

**What**:
- Record named project outcomes alongside final repository validation
results.
- Preserve repository-only validation for deployments without project
attribution.

**Why**:
- Enable consumers to query durable project failures when an integration
supplies attribution.

## Test Plan

- [x] Add unit tests.

## Revert Plan

- Revert this PR. This removes named project-result persistence while
preserving repository validation.

## Issues
-
[CODEM-529](https://linear.app/uber/issue/CODEM-529/persist-project-validation-results)

This branch was successfully deployed

1 active deployment
stack-rebase — 3d3290b3 Deployed Sep 24, 2026 by prathshenoy via Rebase Stack #532
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.

2 participants