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

fix(core): Handle scoop aliases and broken(edited,copied) shim #5551

Merged
merged 5 commits into from
Oct 3, 2023

Conversation

HUMORCE
Copy link
Member

@HUMORCE HUMORCE commented Jun 20, 2023

Description

Motivation and Context

scoop-which cannot get path when these case:

  • scoop which <manually creaetd/edited shim missing shimTarget>
  • scoop which <copied shim> e.g. moving Scoop root directory from other locations
  • scoop which scoop-<scoop_alias>

How Has This Been Tested?

Screenshot_230620033720

Checklist:

  • I have read the Contributing Guide.
  • I have ensured that I am targeting the develop branch.
  • I have updated the documentation accordingly.
  • I have updated the tests accordingly.
  • I have added an entry in the CHANGELOG.

@rashil2000
Copy link
Member

Looks good, update changelog?

@HUMORCE
Copy link
Member Author

HUMORCE commented Oct 3, 2023

Done.

CHANGELOG.md Outdated Show resolved Hide resolved
@rashil2000
Copy link
Member

Also there are conflicts.

HUMORCE and others added 2 commits October 3, 2023 13:20
Co-authored-by: Rashil Gandhi <46838874+rashil2000@users.noreply.github.com>
@rashil2000 rashil2000 merged commit 6d79d62 into ScoopInstaller:develop Oct 3, 2023
2 checks passed
CrendKing pushed a commit to CrendKing/Scoop that referenced this pull request Oct 3, 2023
…pInstaller#5551)

* fix(core): Avoid error messages when shim broked, Allow get path for scoop aliases

* update `scoop-which`

* Add CHANGELOG entry

* Update CHANGELOG.md

Co-authored-by: Rashil Gandhi <46838874+rashil2000@users.noreply.github.com>

---------

Co-authored-by: Rashil Gandhi <46838874+rashil2000@users.noreply.github.com>
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.

None yet

2 participants