Skip to content

Adding reset function based on issue - #413

Open
thebitguru wants to merge 1 commit into
jmoiron:masterfrom
HealthDecision:thebitguru/reset-cache
Open

thebitguru wants to merge 1 commit into
jmoiron:masterfrom
HealthDecision:thebitguru/reset-cache

Conversation

@thebitguru

Copy link
Copy Markdown

Re: #380 Here is a pull request to add a function that resets the cache so multiple result sets can allow StructScan. This is based on @g-rad's approach.

@coveralls

coveralls commented May 18, 2018 •

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 24

  • 0 of 3 (0.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.1%) to 71.676%

Changes Missing Coverage Covered Lines Changed/Added Lines %
sqlx.go 0 3 0.0%
Totals Coverage Status
Change from base Build 17: -0.1%
Covered Lines: 1078
Relevant Lines: 1504

💛 - Coveralls

@glebteterin

Copy link
Copy Markdown

Ping @jmoiron

@ShoshinNikita

Copy link
Copy Markdown

ping @jmoiron

@dlsniper dlsniper added the requires attention The PR looks like it could potentially break things. Definitely requires more testing label Feb 1, 2024
@dlsniper

dlsniper commented Feb 1, 2024

Copy link
Copy Markdown
Collaborator

Hello, @ardan-bkennedy, and I recently stepped in to help maintain this project.
We are sorting the opened issues and pull requests and from what I can see, this might be better served by other opened PRs. When we'll start merging them, this should be sorted out by automatically resetting the cache thanks to those.
Thank you for your contribution.

@mrj0

mrj0 commented Mar 20, 2025

Copy link
Copy Markdown

I think this pr could close in favor of #747 since it includes tests for pg and mysql

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

requires attention The PR looks like it could potentially break things. Definitely requires more testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants