Skip to content

fix(decalrative): change default declarative/ folder to database/#5057

Merged
avallete merged 5 commits intodevelopfrom
fix/rename-default-declarative-dir-to-database
Apr 10, 2026
Merged

fix(decalrative): change default declarative/ folder to database/#5057
avallete merged 5 commits intodevelopfrom
fix/rename-default-declarative-dir-to-database

Conversation

@avallete
Copy link
Copy Markdown
Member

@avallete avallete commented Apr 9, 2026

What kind of change does this PR introduce?

  • Switch the declarative/ default folder name to database/ for better semantic

@avallete avallete requested a review from a team as a code owner April 9, 2026 09:37
@coveralls
Copy link
Copy Markdown

coveralls commented Apr 9, 2026

Coverage Report for CI Build 24238193245

Warning

No base build found for commit a362ed7 on develop.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 63.786%

Details

  • Patch coverage: 11 uncovered changes across 1 file (0 of 11 lines covered, 0.0%).

Uncovered Changes

File Changed Covered %
cmd/db_schema_declarative.go 11 0 0.0%

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 15356
Covered Lines: 9795
Line Coverage: 63.79%
Coverage Strength: 7.02 hits per line

💛 - Coveralls

@avallete avallete changed the title fix(decalrative): change default declarative/ folder to database/ fix(decalrative): change default declarative/ folder to schemas/ Apr 9, 2026
@avallete avallete changed the title fix(decalrative): change default declarative/ folder to schemas/ fix(decalrative): change default declarative/ folder to database/ Apr 9, 2026
if utils.Config.Experimental.PgDelta.Enabled {
viper.Set("EXPERIMENTAL", true)
}
return cmd.Root().PersistentPreRunE(cmd, args)
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note

This needs to be called so the telemetry can capture this command runs.

@avallete avallete merged commit 2faacf1 into develop Apr 10, 2026
20 checks passed
@avallete avallete deleted the fix/rename-default-declarative-dir-to-database branch April 10, 2026 10:50
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.

3 participants