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

exec: do not exclude aliases whet they are specified in --with #731

Merged
merged 4 commits into from
Nov 22, 2021

Conversation

gugod
Copy link
Owner

@gugod gugod commented Nov 20, 2021

The reason why we exclude aliases in exec is to avoid duplicate
run. However, if users explicitly specify aliasses in --with,
skipping those would be counter-intuitive.

PR: #725

The reason why we exclude aliases in `exec` is to avoid duplicate
run. However, if users explicitly specify aliasses in `--with`,
skipping those would be counter-intuitive.

PR: #725
@gugod gugod mentioned this pull request Nov 20, 2021
@coveralls
Copy link

coveralls commented Nov 20, 2021

Coverage Status

Coverage remained the same at 77.457% when pulling 04f1251 on exec-run-with-alias-when-specified into 6073394 on develop.

@gugod gugod merged commit 869daa9 into develop Nov 22, 2021
@gugod gugod deleted the exec-run-with-alias-when-specified branch November 22, 2021 13:18
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