-
Notifications
You must be signed in to change notification settings - Fork 3
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
WFPREV-259 Change project entity to use ObjectiveTypeCodeModel for JPA #427
Conversation
ssylver93
commented
Jan 17, 2025
- Change project entity to use ObjectiveTypeCodeModel
- Fix bug with existing entity code values being set for project update
- Github actions CI/CD change to run latest liquibase changes on build
…eature/WFPREV-259
…eature/WFPREV-259
…eature/WFPREV-259
…eature/WFPREV-259
import ca.bc.gov.nrs.wfprev.data.models.GeneralScopeCodeModel; | ||
import ca.bc.gov.nrs.wfprev.data.models.ProjectModel; | ||
import ca.bc.gov.nrs.wfprev.data.models.ProjectTypeCodeModel; | ||
import ca.bc.gov.nrs.wfprev.data.entities.*; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we generally want to avoid wildcard imports (I think I've been guilty of doing this myself). I'm beting your intellij setting is doing this. YOu can turn up the number of imports much higher than the default that it allows before it starts converting to use imports.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One very minor comment. not worth blocking merge on. Just update your intellij settings for future PRs
Quality Gate passedIssues Measures |