Skip to content

Make non-null the default: use @Nullable as explicit opt-in#22

Merged
stijnvanbael merged 7 commits intomainfrom
copilot/task-3-make-not-null-default
Apr 5, 2026
Merged

Make non-null the default: use @Nullable as explicit opt-in#22
stijnvanbael merged 7 commits intomainfrom
copilot/task-3-make-not-null-default

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 3, 2026

  • Analyse CI build failure: DbMigrationWriterTest.notNullConstraintsAreGeneratedForNonNullableFields failed due to extra trailing \n\n in generated SQL
  • Fix expected SQL file: add extra trailing newline to match processor output
  • Rebase/merge origin/main (auto-generate DB indexes feature) into branch
  • Resolve merge conflict in DbMigrationWriterTest.java: combined both index tests (from main) and notNullConstraints test (from branch)

Copilot AI and others added 2 commits April 3, 2026 10:51
Copilot AI changed the title [WIP] Start working on task 3 to make not null the default Make non-null the default: use @Nullable as explicit opt-in Apr 3, 2026
Copilot AI requested a review from stijnvanbael April 3, 2026 10:55
…istry opt-in

Agent-Logs-Url: https://github.com/stijnvanbael/prefab/sessions/0df0e477-afd9-4710-a3aa-4b374fe5d8de

Co-authored-by: stijnvanbael <1050148+stijnvanbael@users.noreply.github.com>
@stijnvanbael stijnvanbael marked this pull request as ready for review April 4, 2026 07:05
…straints

Agent-Logs-Url: https://github.com/stijnvanbael/prefab/sessions/6925d7b3-fce7-4bde-8ca4-8f6b65de90bd

Co-authored-by: stijnvanbael <1050148+stijnvanbael@users.noreply.github.com>
Co-authored-by: stijnvanbael <1050148+stijnvanbael@users.noreply.github.com>
@stijnvanbael stijnvanbael merged commit 17bf6ae into main Apr 5, 2026
1 check passed
@stijnvanbael stijnvanbael deleted the copilot/task-3-make-not-null-default branch April 5, 2026 16:30
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