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

improve install target handling logic #8498

Merged
merged 4 commits into from
Jan 4, 2023
Merged

Conversation

gbaz
Copy link
Collaborator

@gbaz gbaz commented Sep 28, 2022

Resolves #8325 and resolves #8094.

The latter issue was just a typo. The former issue is subtle and has to do with using "--ignore-project" to differentiate between when we want to parse "foo:bar" as a uri vs. a target selector.

Note that I think this is a strict improvement but still pretty weird. I also added a few comments to help explain what's going on.

@Mikolaj
Copy link
Member

Mikolaj commented Oct 20, 2022

@gbaz: could you rebase (there seem to be a conflict)?

@andreasabel andreasabel added squash+merge me Tell Mergify Bot to squash-merge and removed attention: needs-review labels Jan 2, 2023
@mergify mergify bot added the merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days label Jan 4, 2023
@mergify mergify bot merged commit 062ca76 into master Jan 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days squash+merge me Tell Mergify Bot to squash-merge
Projects
None yet
4 participants