Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: [#312] orm connection is not working #354

Merged
merged 5 commits into from
Nov 28, 2023
Merged

fix: [#312] orm connection is not working #354

merged 5 commits into from
Nov 28, 2023

Conversation

hwbrzzl
Copy link
Contributor

@hwbrzzl hwbrzzl commented Nov 24, 2023

Closes goravel/goravel#312

📑 Description

Solve three problems:

  1. Framework can't identify the connection of slice model;
  2. Framework can't switch connections when the driver is the same;
  3. The conditions will be chaotic when switching connections through the old logic;

✅ Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

ℹ Additional Information

@hwbrzzl hwbrzzl marked this pull request as draft November 24, 2023 00:21
Copy link

codecov bot commented Nov 26, 2023

Codecov Report

Attention: 530 lines in your changes are missing coverage. Please review.

Comparison is base (ad61d31) 62.24% compared to head (a5b128e) 70.69%.
Report is 90 commits behind head on v1.13.x.

Files Patch % Lines
database/gorm/query.go 82.90% 67 Missing and 39 partials ⚠️
database/gorm/cursor.go 36.44% 65 Missing and 3 partials ⚠️
database/console/seed_command.go 35.93% 35 Missing and 6 partials ⚠️
foundation/container.go 15.55% 38 Missing ⚠️
foundation/application.go 41.17% 22 Missing and 8 partials ⚠️
database/gorm/test_models.go 84.06% 20 Missing and 9 partials ⚠️
database/factory.go 68.60% 18 Missing and 9 partials ⚠️
database/gorm/test_utils.go 91.97% 12 Missing and 3 partials ⚠️
log/formatter/general.go 76.56% 11 Missing and 4 partials ⚠️
database/seeder.go 72.00% 8 Missing and 6 partials ⚠️
... and 26 more
Additional details and impacted files
@@             Coverage Diff             @@
##           v1.13.x     #354      +/-   ##
===========================================
+ Coverage    62.24%   70.69%   +8.45%     
===========================================
  Files          125      145      +20     
  Lines         8078    10138    +2060     
===========================================
+ Hits          5028     7167    +2139     
+ Misses        2717     2522     -195     
- Partials       333      449     +116     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor Author

@hwbrzzl hwbrzzl Nov 27, 2023

Choose a reason for hiding this comment

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

Solve the problem: The conditions will be chaotic when switching connections through the old logic.

Before, the Where method, etc. calls the Gorm method directly and gets a new Gorm instance.

Now, the conditions will be saved in Conditions, and will be built when calling a final method, like: Get.

After this change, we can control the conditions and the Gorm instance more freely.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

There is no changes for this file, just order the methods.

}

func (r *QueryImpl) Association(association string) ormcontract.Association {
return r.instance.Association(association)
query := r.buildConditions()
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The buildConditions method will be called when need to call the Gorm method directly.

@@ -2330,6 +2112,107 @@ func (s *QueryTestSuite) TestRaw() {
}
}

func (s *QueryTestSuite) TestReuse() {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

After this change, we can set query to a variable and reuse it, such as:

query := facades.Orm().Query().Where("a = b")
query.Where("b = c").Find() == facades.Orm().Query().Where("a = b").Where("b = c").Find()
query.Where("c = d").Find() == facades.Orm().Query().Where("a = b").Where("c = d").Find()

return nil
}

func (r *QueryImpl) new(db *gormio.DB) *QueryImpl {
Copy link
Contributor Author

@hwbrzzl hwbrzzl Nov 27, 2023

Choose a reason for hiding this comment

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

This is the key that query can be reused: the conditions will be copied when calling conditions methods.

return query.(*QueryImpl).instance
},
}
func (r *QueryImpl) buildConditions() *QueryImpl {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The original Gorm instance of QueryImpl will not be changed, to avoid the conditions chaotic.

@@ -1017,6 +1360,37 @@ func filterFindConditions(conds ...any) error {
return nil
}

func getModelConnection(model any) (string, error) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fix:

  1. Framework can't identify the connection of slice model;
  2. Framework can't switch connections when the driver is the same;

@hwbrzzl hwbrzzl marked this pull request as ready for review November 27, 2023 09:13
@hwbrzzl hwbrzzl requested a review from a team November 27, 2023 09:14
Copy link
Member

@devhaozi devhaozi left a comment

Choose a reason for hiding this comment

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

LGTM

@hwbrzzl hwbrzzl merged commit 19331ce into v1.13.x Nov 28, 2023
11 checks passed
@hwbrzzl hwbrzzl deleted the bowen/#312 branch November 28, 2023 07:39
hwbrzzl added a commit that referenced this pull request Feb 22, 2024
* Upgrade v1.13.1

* Merge #305, #306 to v1.13.x (#308)

* Upgrade v1.13.1 (#295)

* Update README

* build(deps): bump the go-modules group with 1 update (#296)

Bumps the go-modules group with 1 update: [github.com/bytedance/sonic](https://github.com/bytedance/sonic).

- [Release notes](https://github.com/bytedance/sonic/releases)
- [Commits](bytedance/sonic@v1.10.0...v1.10.1)

---
updated-dependencies:
- dependency-name: github.com/bytedance/sonic
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: go-modules
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump the go-modules group with 1 update (#297)

Bumps the go-modules group with 1 update: [github.com/golang-module/carbon/v2](https://github.com/golang-module/carbon).

- [Release notes](https://github.com/golang-module/carbon/releases)
- [Commits](golang-module/carbon@v2.2.6...v2.2.7)

---
updated-dependencies:
- dependency-name: github.com/golang-module/carbon/v2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: go-modules
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump the go-modules group with 1 update (#299)

Bumps the go-modules group with 1 update: [google.golang.org/grpc](https://github.com/grpc/grpc-go).

- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.58.0...v1.58.1)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: go-modules
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump the github-actions group with 1 update (#300)

Bumps the github-actions group with 1 update: [codecov/codecov-action](https://github.com/codecov/codecov-action).

- [Release notes](https://github.com/codecov/codecov-action/releases)
- [Changelog](https://github.com/codecov/codecov-action/blob/main/CHANGELOG.md)
- [Commits](codecov/codecov-action@v3...v4)

---
updated-dependencies:
- dependency-name: codecov/codecov-action
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: github-actions
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Wenbo Han <[email protected]>

* feat: add GetBytes method and test cases (#298)

* feat: add GetBytes method and test cases

* workflow: remove filesystem

* fix: lint

* feat: optimize code

* feat: optimize test

* fix: windows tests

* workflow: fix

* fix: file not close

* feat: optimize code

* fix: lint

* fix: lint

* fix: test

* feat: Add interface comments (#292)

* Add interface comments to all facades

* Add comments for remaining contracts

* convert the first letter of word to a lower case in godoc comments

* Add a space for each comment after //

* build(deps): bump the go-modules group with 1 update (#301)

Bumps the go-modules group with 1 update: [github.com/golang-module/carbon/v2](https://github.com/golang-module/carbon).

- [Release notes](https://github.com/golang-module/carbon/releases)
- [Commits](golang-module/carbon@v2.2.7...v2.2.8)

---
updated-dependencies:
- dependency-name: github.com/golang-module/carbon/v2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: go-modules
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump the go-modules group with 1 update (#303)

Bumps the go-modules group with 1 update: [google.golang.org/grpc](https://github.com/grpc/grpc-go).

- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.58.1...v1.58.2)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: go-modules
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix:[#263] There is a path error in the filesystem module (#305)

* feat: Add Fluent String helpers (#304)

* add fluent strings helpers

* add more fluent string helpers

* add headline helper

* add test cases for fluent strings

* add test cases for headline function

* modify, according to suggestions

* modify, according to suggestions

* reformat the str file

* fix: #267 (#306)

* feat: add environment helper

* feat: optimize sonic

* fix: tests

* feat: rename to env

* feat: optimize json

* fix: optimize tests

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: 耗子 <[email protected]>
Co-authored-by: krishan kumar <[email protected]>

* fix: [#270] Optimize postgres DNS (#309)

* fix: [#270] Optimize postgres DNS

* Fix unit test

* fix: [#253] [Bug] The With method can't be used with the Cursor method together (#311)

* fix: #253 [Bug] The With method can't be used with the Cursor method together

* Remove test

* Upgrade v1.13.2

* Update README

* fix: Register multiple events (#317)

* Upgrade v1.13.1 (#295)

* Update README

* build(deps): bump the go-modules group with 1 update (#296)

Bumps the go-modules group with 1 update: [github.com/bytedance/sonic](https://github.com/bytedance/sonic).

- [Release notes](https://github.com/bytedance/sonic/releases)
- [Commits](bytedance/sonic@v1.10.0...v1.10.1)

---
updated-dependencies:
- dependency-name: github.com/bytedance/sonic
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: go-modules
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump the go-modules group with 1 update (#297)

Bumps the go-modules group with 1 update: [github.com/golang-module/carbon/v2](https://github.com/golang-module/carbon).

- [Release notes](https://github.com/golang-module/carbon/releases)
- [Commits](golang-module/carbon@v2.2.6...v2.2.7)

---
updated-dependencies:
- dependency-name: github.com/golang-module/carbon/v2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: go-modules
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump the go-modules group with 1 update (#299)

Bumps the go-modules group with 1 update: [google.golang.org/grpc](https://github.com/grpc/grpc-go).

- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.58.0...v1.58.1)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: go-modules
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump the github-actions group with 1 update (#300)

Bumps the github-actions group with 1 update: [codecov/codecov-action](https://github.com/codecov/codecov-action).

- [Release notes](https://github.com/codecov/codecov-action/releases)
- [Changelog](https://github.com/codecov/codecov-action/blob/main/CHANGELOG.md)
- [Commits](codecov/codecov-action@v3...v4)

---
updated-dependencies:
- dependency-name: codecov/codecov-action
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: github-actions
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Wenbo Han <[email protected]>

* feat: add GetBytes method and test cases (#298)

* feat: add GetBytes method and test cases

* workflow: remove filesystem

* fix: lint

* feat: optimize code

* feat: optimize test

* fix: windows tests

* workflow: fix

* fix: file not close

* feat: optimize code

* fix: lint

* fix: lint

* fix: test

* feat: Add interface comments (#292)

* Add interface comments to all facades

* Add comments for remaining contracts

* convert the first letter of word to a lower case in godoc comments

* Add a space for each comment after //

* build(deps): bump the go-modules group with 1 update (#301)

Bumps the go-modules group with 1 update: [github.com/golang-module/carbon/v2](https://github.com/golang-module/carbon).

- [Release notes](https://github.com/golang-module/carbon/releases)
- [Commits](golang-module/carbon@v2.2.7...v2.2.8)

---
updated-dependencies:
- dependency-name: github.com/golang-module/carbon/v2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: go-modules
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump the go-modules group with 1 update (#303)

Bumps the go-modules group with 1 update: [google.golang.org/grpc](https://github.com/grpc/grpc-go).

- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.58.1...v1.58.2)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: go-modules
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix:[#263] There is a path error in the filesystem module (#305)

* feat: Add Fluent String helpers (#304)

* add fluent strings helpers

* add more fluent string helpers

* add headline helper

* add test cases for fluent strings

* add test cases for headline function

* modify, according to suggestions

* modify, according to suggestions

* reformat the str file

* fix: #267 (#306)

* feat: add environment helper

* feat: optimize sonic

* fix: tests

* feat: rename to env

* feat: optimize json

* fix: optimize tests

* refactor: move mocks (#307)

* feat: update mocks

* refactor: move mocks

* workflow: remove code review

* workflow: fix mockery

* fix: update mocks

* refactor: move gorm initialize interface to contracts folder

* feat: test generate mock

* workflow: update mock

* chore: update mocks

* chore: remove unnecessary go generate tag

* workflow: only generate mocks in master branch

* workflow: fix lint

* workflow: make mockery only run on pr

* feat: move Gorm interface to contracts/database/gorm

* feat: move Gorm interface to contracts/database/gorm

* feat: move interface

* chore: update mocks

* fix: test

---------

Co-authored-by: devhaozi <[email protected]>

* build(deps): bump the go-modules group with 1 update (#310)

Bumps the go-modules group with 1 update: [golang.org/x/crypto](https://github.com/golang/crypto).

- [Commits](golang/crypto@v0.13.0...v0.14.0)

---
updated-dependencies:
- dependency-name: golang.org/x/crypto
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: go-modules
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump the github-actions group with 1 update (#312)

Bumps the github-actions group with 1 update: [stefanzweifel/git-auto-commit-action](https://github.com/stefanzweifel/git-auto-commit-action).

- [Release notes](https://github.com/stefanzweifel/git-auto-commit-action/releases)
- [Changelog](https://github.com/stefanzweifel/git-auto-commit-action/blob/master/CHANGELOG.md)
- [Commits](stefanzweifel/git-auto-commit-action@v4...v5)

---
updated-dependencies:
- dependency-name: stefanzweifel/git-auto-commit-action
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: github-actions
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump the go-modules group with 1 update (#313)

Bumps the go-modules group with 1 update: [github.com/spf13/viper](https://github.com/spf13/viper).

- [Release notes](https://github.com/spf13/viper/releases)
- [Commits](spf13/viper@v1.16.0...v1.17.0)

---
updated-dependencies:
- dependency-name: github.com/spf13/viper
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: go-modules
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Wenbo Han <[email protected]>

* build(deps): bump the go-modules group with 2 updates (#315)

Bumps the go-modules group with 2 updates: [github.com/bytedance/sonic](https://github.com/bytedance/sonic) and [github.com/golang-module/carbon/v2](https://github.com/golang-module/carbon).


Updates `github.com/bytedance/sonic` from 1.10.1 to 1.10.2
- [Release notes](https://github.com/bytedance/sonic/releases)
- [Commits](bytedance/sonic@v1.10.1...v1.10.2)

Updates `github.com/golang-module/carbon/v2` from 2.2.8 to 2.2.9
- [Release notes](https://github.com/golang-module/carbon/releases)
- [Commits](golang-module/carbon@v2.2.8...v2.2.9)

---
updated-dependencies:
- dependency-name: github.com/bytedance/sonic
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: go-modules
- dependency-name: github.com/golang-module/carbon/v2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: go-modules
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Register multiple events

* Fix typo

* Restore

* Allow multiple events registeration

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Wenbo Han <[email protected]>
Co-authored-by: Bowen <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: 耗子 <[email protected]>
Co-authored-by: krishan kumar <[email protected]>
Co-authored-by: devhaozi <[email protected]>

* fix: database variable (#316)

* Upgrade v1.13.1 (#295)

* Update README

* build(deps): bump the go-modules group with 1 update (#296)

Bumps the go-modules group with 1 update: [github.com/bytedance/sonic](https://github.com/bytedance/sonic).

- [Release notes](https://github.com/bytedance/sonic/releases)
- [Commits](bytedance/sonic@v1.10.0...v1.10.1)

---
updated-dependencies:
- dependency-name: github.com/bytedance/sonic
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: go-modules
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump the go-modules group with 1 update (#297)

Bumps the go-modules group with 1 update: [github.com/golang-module/carbon/v2](https://github.com/golang-module/carbon).

- [Release notes](https://github.com/golang-module/carbon/releases)
- [Commits](golang-module/carbon@v2.2.6...v2.2.7)

---
updated-dependencies:
- dependency-name: github.com/golang-module/carbon/v2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: go-modules
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump the go-modules group with 1 update (#299)

Bumps the go-modules group with 1 update: [google.golang.org/grpc](https://github.com/grpc/grpc-go).

- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.58.0...v1.58.1)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: go-modules
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump the github-actions group with 1 update (#300)

Bumps the github-actions group with 1 update: [codecov/codecov-action](https://github.com/codecov/codecov-action).

- [Release notes](https://github.com/codecov/codecov-action/releases)
- [Changelog](https://github.com/codecov/codecov-action/blob/main/CHANGELOG.md)
- [Commits](codecov/codecov-action@v3...v4)

---
updated-dependencies:
- dependency-name: codecov/codecov-action
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: github-actions
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Wenbo Han <[email protected]>

* feat: add GetBytes method and test cases (#298)

* feat: add GetBytes method and test cases

* workflow: remove filesystem

* fix: lint

* feat: optimize code

* feat: optimize test

* fix: windows tests

* workflow: fix

* fix: file not close

* feat: optimize code

* fix: lint

* fix: lint

* fix: test

* feat: Add interface comments (#292)

* Add interface comments to all facades

* Add comments for remaining contracts

* convert the first letter of word to a lower case in godoc comments

* Add a space for each comment after //

* build(deps): bump the go-modules group with 1 update (#301)

Bumps the go-modules group with 1 update: [github.com/golang-module/carbon/v2](https://github.com/golang-module/carbon).

- [Release notes](https://github.com/golang-module/carbon/releases)
- [Commits](golang-module/carbon@v2.2.7...v2.2.8)

---
updated-dependencies:
- dependency-name: github.com/golang-module/carbon/v2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: go-modules
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump the go-modules group with 1 update (#303)

Bumps the go-modules group with 1 update: [google.golang.org/grpc](https://github.com/grpc/grpc-go).

- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.58.1...v1.58.2)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: go-modules
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix:[#263] There is a path error in the filesystem module (#305)

* feat: Add Fluent String helpers (#304)

* add fluent strings helpers

* add more fluent string helpers

* add headline helper

* add test cases for fluent strings

* add test cases for headline function

* modify, according to suggestions

* modify, according to suggestions

* reformat the str file

* fix: #267 (#306)

* feat: add environment helper

* feat: optimize sonic

* fix: tests

* feat: rename to env

* feat: optimize json

* fix: optimize tests

* refactor: move mocks (#307)

* feat: update mocks

* refactor: move mocks

* workflow: remove code review

* workflow: fix mockery

* fix: update mocks

* refactor: move gorm initialize interface to contracts folder

* feat: test generate mock

* workflow: update mock

* chore: update mocks

* chore: remove unnecessary go generate tag

* workflow: only generate mocks in master branch

* workflow: fix lint

* workflow: make mockery only run on pr

* feat: move Gorm interface to contracts/database/gorm

* feat: move Gorm interface to contracts/database/gorm

* feat: move interface

* chore: update mocks

* fix: test

---------

Co-authored-by: devhaozi <[email protected]>

* build(deps): bump the go-modules group with 1 update (#310)

Bumps the go-modules group with 1 update: [golang.org/x/crypto](https://github.com/golang/crypto).

- [Commits](golang/crypto@v0.13.0...v0.14.0)

---
updated-dependencies:
- dependency-name: golang.org/x/crypto
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: go-modules
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump the github-actions group with 1 update (#312)

Bumps the github-actions group with 1 update: [stefanzweifel/git-auto-commit-action](https://github.com/stefanzweifel/git-auto-commit-action).

- [Release notes](https://github.com/stefanzweifel/git-auto-commit-action/releases)
- [Changelog](https://github.com/stefanzweifel/git-auto-commit-action/blob/master/CHANGELOG.md)
- [Commits](stefanzweifel/git-auto-commit-action@v4...v5)

---
updated-dependencies:
- dependency-name: stefanzweifel/git-auto-commit-action
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: github-actions
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump the go-modules group with 1 update (#313)

Bumps the go-modules group with 1 update: [github.com/spf13/viper](https://github.com/spf13/viper).

- [Release notes](https://github.com/spf13/viper/releases)
- [Commits](spf13/viper@v1.16.0...v1.17.0)

---
updated-dependencies:
- dependency-name: github.com/spf13/viper
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: go-modules
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Wenbo Han <[email protected]>

* build(deps): bump the go-modules group with 2 updates (#315)

Bumps the go-modules group with 2 updates: [github.com/bytedance/sonic](https://github.com/bytedance/sonic) and [github.com/golang-module/carbon/v2](https://github.com/golang-module/carbon).


Updates `github.com/bytedance/sonic` from 1.10.1 to 1.10.2
- [Release notes](https://github.com/bytedance/sonic/releases)
- [Commits](bytedance/sonic@v1.10.1...v1.10.2)

Updates `github.com/golang-module/carbon/v2` from 2.2.8 to 2.2.9
- [Release notes](https://github.com/golang-module/carbon/releases)
- [Commits](golang-module/carbon@v2.2.8...v2.2.9)

---
updated-dependencies:
- dependency-name: github.com/bytedance/sonic
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: go-modules
- dependency-name: github.com/golang-module/carbon/v2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: go-modules
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Fix database variable

* Restore

* Fix database typo

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Wenbo Han <[email protected]>
Co-authored-by: Bowen <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: 耗子 <[email protected]>
Co-authored-by: krishan kumar <[email protected]>
Co-authored-by: devhaozi <[email protected]>

* feat: [#265] The Config module supports get system environment variables (#329)

* feat: [#265] The Config module supports get system environment variables (#335)

* feat: [#265] The Config module supports get system environment variables

* Add unit tests

* Upgrade v1.13.3

* Update mock

* Upgrade v1.13.4

* fix: [#312] orm connection is not working (#354)

* fix: [#312] orm connection is not working

* Optimize logic

* Optimize unit test

* Optimize unit test

* Optimize unit tests

---------

Co-authored-by: Bowen <[email protected]>

* fix: [#320] mock.Log() is not working (#355)

* Upgrade v1.13.5 (#357)

* fix: [#341] The Chain Query isn't executed one by one (#380)

* Upgrade v1.13.7

* Merge orm

* Optmize merge

* update go.mod

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: 耗子 <[email protected]>
Co-authored-by: krishan kumar <[email protected]>
Co-authored-by: Ahmed M. Ammar <[email protected]>
Co-authored-by: devhaozi <[email protected]>
Co-authored-by: Bowen <[email protected]>
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.

2 participants