Skip to content

Add Danger checks to ensure db changes are added#396

Merged
r0adkll merged 1 commit intomainfrom
dh/danger-database-checks
Oct 29, 2025
Merged

Add Danger checks to ensure db changes are added#396
r0adkll merged 1 commit intomainfrom
dh/danger-database-checks

Conversation

@r0adkll
Copy link
Owner

@r0adkll r0adkll commented Oct 29, 2025

Danger should check if DB schema changes were made, migrations were added, and the code to migrate was updated

@github-actions
Copy link

github-actions bot commented Oct 29, 2025

Warnings
⚠️

Any changes to library code should be reflected in the Changelog.

Please consider adding a note there and adhere to the Changelog Guidelines.

⚠️

Ignoring DB schema changes due to ignore-db-change label

Messages
📖 This PR has been checked by Danger

Generated by 🚫 Danger Kotlin against 941e549

@r0adkll r0adkll added the ignore-db-change Used to tell Danger to ignore .sq changes as they only pertain to queries and not schema changes. label Oct 29, 2025
@r0adkll r0adkll force-pushed the dh/danger-database-checks branch from 3c25ccc to 941e549 Compare October 29, 2025 21:08
@r0adkll r0adkll removed the ignore-db-change Used to tell Danger to ignore .sq changes as they only pertain to queries and not schema changes. label Oct 29, 2025
@r0adkll r0adkll enabled auto-merge (squash) October 29, 2025 21:09
@github-actions
Copy link

github-actions bot commented Oct 29, 2025

Android APK @ b8d93d2

@r0adkll r0adkll merged commit 1663bab into main Oct 29, 2025
5 checks passed
@r0adkll r0adkll deleted the dh/danger-database-checks branch October 29, 2025 21:47
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