Revert "Add Release::asset_for ability to find asset byOS and ARCH"
#164
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #136
Reverts #123
In its current state, this is a breaking change.
There are two ways to solve #136:
1. Revert #123
This is the current approach this PR takes. In my opinion, the full target should just be in the filename. This is not
cargo-binstall; it's totally valid to have a requirement for the filename format when usingself_update.2. Fix logic, prioritize target
Quoting from #136:
This is also fine, and probably what @DervexDev intended to do in #123 in the first place, given the way the changelog is written. If you want to take this approach instead, let me know and I'll edit the PR.