Skip to content

danger: Require attestation that database schema migrations have been applied - #58469

Merged
maxdeviant merged 4 commits into
mainfrom
push-mvrprmkzlmkl
Jun 3, 2026
Merged

danger: Require attestation that database schema migrations have been applied#58469
maxdeviant merged 4 commits into
mainfrom
push-mvrprmkzlmkl

Conversation

@maxdeviant

@maxdeviant maxdeviant commented Jun 3, 2026

Copy link
Copy Markdown
Member

This PR updates the Danger check for touching the Collab schema files to require attestation that the database schema migrations have been created and applied.

If there are changes to the schema files without the attestation, Danger will fail the status check with an error:

Screenshot 2026-06-03 at 4 58 56 PM

When the attestation clause is present, Danger will report it as such:

Screenshot 2026-06-03 at 4 59 45 PM

Release Notes:

  • N/A

@maxdeviant maxdeviant self-assigned this Jun 3, 2026
@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Jun 3, 2026
@zed-community-bot zed-community-bot Bot added the staff Pull requests authored by a current member of Zed staff label Jun 3, 2026
@maxdeviant
maxdeviant requested a review from MrSubidubi June 3, 2026 21:14
@maxdeviant
maxdeviant added this pull request to the merge queue Jun 3, 2026
Merged via the queue into main with commit 7e42c0b Jun 3, 2026
33 checks passed
@maxdeviant
maxdeviant deleted the push-mvrprmkzlmkl branch June 3, 2026 21:21
@zed-zippy zed-zippy Bot added the PR state:needs review Used to label PRs that are in need of a post-merge approval label Jun 3, 2026
@MrSubidubi

Copy link
Copy Markdown
Member

@zed-industries/approved

@zed-zippy zed-zippy Bot removed the PR state:needs review Used to label PRs that are in need of a post-merge approval label Jun 3, 2026
TomPlanche pushed a commit to TomPlanche/zed that referenced this pull request Jun 8, 2026
… applied (zed-industries#58469)

This PR updates the Danger check for touching the Collab schema files to
require attestation that the database schema migrations have been
created and applied.

If there are changes to the schema files without the attestation, Danger
will fail the status check with an error:

<img width="959" height="635" alt="Screenshot 2026-06-03 at 4 58 56 PM"
src="https://github.com/user-attachments/assets/e0857137-b351-4212-a023-13a7d53f5934"
/>

When the attestation clause is present, Danger will report it as such:

<img width="901" height="333" alt="Screenshot 2026-06-03 at 4 59 45 PM"
src="https://github.com/user-attachments/assets/098b0ce4-f86e-4d41-b6af-765bc015fe6e"
/>

Release Notes:

- N/A
This was referenced Jun 18, 2026
jonx pushed a commit to jonx/zed-aros that referenced this pull request Jul 17, 2026
… applied (zed-industries#58469)

This PR updates the Danger check for touching the Collab schema files to
require attestation that the database schema migrations have been
created and applied.

If there are changes to the schema files without the attestation, Danger
will fail the status check with an error:

<img width="959" height="635" alt="Screenshot 2026-06-03 at 4 58 56 PM"
src="https://github.com/user-attachments/assets/e0857137-b351-4212-a023-13a7d53f5934"
/>

When the attestation clause is present, Danger will report it as such:

<img width="901" height="333" alt="Screenshot 2026-06-03 at 4 59 45 PM"
src="https://github.com/user-attachments/assets/098b0ce4-f86e-4d41-b6af-765bc015fe6e"
/>

Release Notes:

- N/A
jolutz pushed a commit to jolutz/zed that referenced this pull request Aug 8, 2026
… applied (zed-industries#58469)

This PR updates the Danger check for touching the Collab schema files to
require attestation that the database schema migrations have been
created and applied.

If there are changes to the schema files without the attestation, Danger
will fail the status check with an error:

<img width="959" height="635" alt="Screenshot 2026-06-03 at 4 58 56 PM"
src="https://github.com/user-attachments/assets/e0857137-b351-4212-a023-13a7d53f5934"
/>

When the attestation clause is present, Danger will report it as such:

<img width="901" height="333" alt="Screenshot 2026-06-03 at 4 59 45 PM"
src="https://github.com/user-attachments/assets/098b0ce4-f86e-4d41-b6af-765bc015fe6e"
/>

Release Notes:

- N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement staff Pull requests authored by a current member of Zed staff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants