Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix enum constraint checks missing from Liquibase changelog #538

Merged
merged 1 commit into from
Jan 30, 2024

Conversation

nscuro
Copy link
Member

@nscuro nscuro commented Jan 24, 2024

Description

Fixes enum constraint checks missing from Liquibase changelog.

Addressed Issue

Fixes DependencyTrack/hyades#1030

Additional Details

addCheckConstraint is a Pro feature of Liquibase, so we have to fall back to raw SQL.

Checklist

  • I have read and understand the contributing guidelines
  • This PR fixes a defect, and I have provided tests to verify that the fix is effective
  • This PR implements an enhancement, and I have provided tests to verify that it works as intended
  • This PR introduces changes to the database model, and I have added corresponding update logic
  • This PR introduces new or alters existing behavior, and I have updated the documentation accordingly

@nscuro nscuro added the defect Something isn't working label Jan 24, 2024
VithikaS
VithikaS previously approved these changes Jan 30, 2024
@VithikaS VithikaS merged commit e1c8435 into main Jan 30, 2024
6 checks passed
@VithikaS VithikaS deleted the issue-1030 branch January 30, 2024 09:47
@nscuro nscuro added this to the 5.3.0 milestone Feb 7, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 9, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
defect Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Liquibase changelog does not cover existing CHECK constraints
2 participants