Skip to content

fix: use restoreOptions instead of restoreBinary for defaultOptions#247

Merged
SimonMellerin merged 1 commit intomakinacorpus:mainfrom
rconstant:fixes/fix-restorer-using-binary-instead-of-options
Jan 23, 2026
Merged

fix: use restoreOptions instead of restoreBinary for defaultOptions#247
SimonMellerin merged 1 commit intomakinacorpus:mainfrom
rconstant:fixes/fix-restorer-using-binary-instead-of-options

Conversation

@rconstant
Copy link
Copy Markdown
Contributor

Hi,

First, thank you for this bundle, it’s been really helpful.

I noticed a small issue when changing the restore options in the db_tools.yml file: the binary was being set instead.
This PR includes a quick fix that should resolve the issue.

Thanks

@pounard
Copy link
Copy Markdown
Member

pounard commented Jan 21, 2026

Nice, thanks! Ping @SimonMellerin

@SimonMellerin SimonMellerin enabled auto-merge (rebase) January 22, 2026 08:16
@SimonMellerin
Copy link
Copy Markdown
Member

Hi @rconstant , thanks for this fix !

Can you rebase your branch on main to fix these phpstan errors ?

Thanks !

auto-merge was automatically disabled January 22, 2026 17:10

Head branch was pushed to by a user without write access

@SimonMellerin SimonMellerin merged commit 65a599b into makinacorpus:main Jan 23, 2026
12 checks passed
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.

3 participants