Skip to content

feat: [#358] Remove driver.Schema#28

Merged
hwbrzzl merged 1 commit intomasterfrom
bowen/#358-21
Mar 9, 2025
Merged

feat: [#358] Remove driver.Schema#28
hwbrzzl merged 1 commit intomasterfrom
bowen/#358-21

Conversation

@hwbrzzl
Copy link
Contributor

@hwbrzzl hwbrzzl commented Mar 9, 2025

📑 Description

goravel/goravel#358

This pull request includes several updates to the grammar.go and grammar_test.go files, as well as a dependency update in the go.mod file. The changes focus on modifying function signatures and updating a dependency version. Here are the most important changes:

Dependency Update:

  • Updated github.com/goravel/framework dependency from version v1.15.2-0.20250308100915-cabc06df5398 to v1.15.2-0.20250309092812-0d45fdd3fb26 in go.mod.

Function Signature Modifications:

  • Modified the CompileRenameColumn function signature in grammar.go to include an additional parameter _ []driver.Column.
  • Modified the CompileRenameIndex function signature in grammar.go to include an additional parameter _ []driver.Index.

Test Updates:

  • Updated the TestCompileRenameColumn test in grammar_test.go to match the new CompileRenameColumn function signature.

✅ Checks

  • Added test cases for my code

@hwbrzzl hwbrzzl requested a review from a team as a code owner March 9, 2025 09:30
@hwbrzzl hwbrzzl merged commit 9f6835a into master Mar 9, 2025
5 checks passed
@hwbrzzl hwbrzzl deleted the bowen/#358-21 branch September 6, 2025 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant