Skip to content

Fix the package update#5129

Merged
TimWolla merged 7 commits intomasterfrom
package-fix-update
Nov 18, 2022
Merged

Fix the package update#5129
TimWolla merged 7 commits intomasterfrom
package-fix-update

Conversation

@TimWolla
Copy link
Copy Markdown
Member

Fixes #5094

As `$this->package` is effectively readonly, this is safe and will not change
the behavior.
…nstructions()`

The filtering of update instructions is a massive design mistake is now
effectively removed. The method is renamed, because the behavior changed quite
a bit. Better fail obviously than silently returning garbage.

Fixes #5094
This property is no longer used.
@TimWolla TimWolla added the Bug label Nov 18, 2022
@TimWolla TimWolla requested a review from dtdesign November 18, 2022 11:07
@TimWolla TimWolla merged commit 496bdfe into master Nov 18, 2022
@TimWolla TimWolla deleted the package-fix-update branch November 18, 2022 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Package update fails

2 participants