Skip to content

chore: minor DX enhancements#3

Merged
graduenz merged 8 commits intomainfrom
gui/improvements
Jan 26, 2026
Merged

chore: minor DX enhancements#3
graduenz merged 8 commits intomainfrom
gui/improvements

Conversation

@graduenz
Copy link
Contributor

@graduenz graduenz commented Jan 25, 2026

Minor developer experience enhancements after project restructure

Renamed some functions:

- ExistsTable
+ HasTable

- ExistsTableColumn
+ HasColumn

Cleaner MigratorConfig construction using function chain:

- config := migrations.NewMigratorConfig(
- 	migrations.WithTableName("custom_migrations"),
- 	migrations.WithColumnNames("migration_name", "applied_on"),
- 	migrations.WithMigrationNameMaxLength(500),
- )

+ config := migrations.NewMigratorConfig().
+ 	WithTableName("custom_migrations").
+ 	WithColumnNames("migration_name", "applied_on").
+ 	WithMigrationNameMaxLength(500)

Cleaner case-insensitive comparison in schema package:

Instead of using for loops and the == comparer with strings.ToLower() or strings.ToUpper(), changed it to use slices.ContainsFunc and strings.EqualFold():

- for _, lTable := range lTables {
- 	lUpperTableName := strings.ToUpper(lTable)
- 	if lUpperTableName == strings.ToUpper(pTableName.String()) {
- 		return true
- 	}
- }
- return false

+ return slices.ContainsFunc(lTables, func(t string) bool {
+ 	return strings.EqualFold(t, pTableName.String())
+ })

@graduenz graduenz changed the title Gui/improvements chore: minor DX enhancements Jan 26, 2026
@sonarqubecloud
Copy link

@graduenz graduenz merged commit 07a1c0a into main Jan 26, 2026
3 checks passed
@graduenz graduenz deleted the gui/improvements branch January 26, 2026 21:27
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