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

[maint]: a new gh-r test #449

Closed
psprint opened this issue Dec 28, 2022 · 4 comments · Fixed by #455
Closed

[maint]: a new gh-r test #449

psprint opened this issue Dec 28, 2022 · 4 comments · Fixed by #455

Comments

@psprint
Copy link
Contributor

psprint commented Dec 28, 2022

Summary

I saw that there is a group of gh-r tests that download multiple projects. There is one other somewhat interesting project to test: naggie/dstask, because zinit normally chooses wrong "import" binary, however with a clever bpick'' this can be fixed:

zi sbin'dstask*->tsk' bpick'^*import*' ghapi from'gh-r' for dstas

The bpick'' iced blocks "import" binaries and makes zinit choose the main binary, because ^ is a negation.

Proposed fix

No response

@vladdoster
Copy link
Member

vladdoster commented Jan 8, 2023

@psprint What does ghapi do? It errors for me.

 zi lbin'!dstask*->tsk' bpick'^*import*' ghapi from'gh-r' for naggie/dstask
ERROR: Unknown subcommand: `ghapi` It should be one of the following:
1. a subcommand - i.e., `load`, `snippet`, `update`, `delete` (e.g., `zinit load username/reponame`)
2. a for-based command body - i.e., `no compile`, `from'g-r'`, `null`, etc. (e.g., `zinit <ICE(s)> for <PLUGIN/SNIPPET>`) 

See `help` for a more detailed usage information and the list of the subcommands.

Screenshot 2023-01-08 at 15 06 51

@psprint
Copy link
Contributor Author

psprint commented Jan 8, 2023

@vladdoster: ah, I've forgot to clear unscope annex.

zi  from'gh-r' sbin'dstask*->tsk' bpick'^*import*' for naggie/dstask

@vladdoster
Copy link
Member

@psprint PR open #455

@vladdoster
Copy link
Member

Screenshot 2023-01-08 at 15 31 20

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 a pull request may close this issue.

2 participants