Skip to content

chore: upgrade v1.17#85

Merged
hwbrzzl merged 1 commit intomasterfrom
bowen/upgrade-v1.17-1
Feb 2, 2026
Merged

chore: upgrade v1.17#85
hwbrzzl merged 1 commit intomasterfrom
bowen/upgrade-v1.17-1

Conversation

@hwbrzzl
Copy link
Contributor

@hwbrzzl hwbrzzl commented Feb 2, 2026

Greptile Overview

Greptile Summary

This PR upgrades the github.com/goravel/framework dependency to align with v1.17 API changes. The framework renamed provider registration methods for better clarity: AddProviderApplyRegisterProvider and RemoveProviderApplyUnregisterProvider.

Key Changes:

  • Updated framework dependency from snapshot v1.16.1-0.20260128144145-280cdc17fcee to v1.16.1-0.20260202061217-60e5444dde33
  • Renamed method calls in setup/setup.go to match the new framework API
  • Updated go.sum checksums accordingly

The changes are straightforward API renames with identical functionality. The setup script continues to handle both bootstrap and traditional setup modes correctly.

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • The changes are minimal and mechanical - simple method renames to match the updated framework API. The logic remains identical, and the changes follow the framework's breaking changes correctly.
  • No files require special attention

Important Files Changed

Filename Overview
go.mod Updated github.com/goravel/framework dependency from 2026-01-28 to 2026-02-02 snapshot version
go.sum Updated checksums for the new framework version
setup/setup.go Renamed AddProviderApply to RegisterProvider and RemoveProviderApply to UnregisterProvider to match framework v1.17 API

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant Setup as setup/setup.go
    participant Framework as goravel/framework
    participant Bootstrap as Bootstrap Setup
    participant App as App Config

    Dev->>Setup: Run package install/uninstall
    
    alt Bootstrap Setup Enabled
        Setup->>Framework: Call RegisterProvider()
        Framework->>Bootstrap: Add provider to bootstrap/providers.go
        Note over Framework,Bootstrap: New API name in v1.17
    else Traditional Setup
        Setup->>Framework: Call Register()
        Framework->>App: Add provider to app.go
    end
    
    Setup->>App: Update database.go config
    
    Note over Setup,Framework: Uninstall process uses UnregisterProvider()<br/>instead of RemoveProviderApply()
Loading

@hwbrzzl hwbrzzl merged commit 5859268 into master Feb 2, 2026
6 checks passed
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