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

Test alr from project dir #1262

Merged
merged 3 commits into from
Dec 13, 2022

Conversation

0rzech
Copy link
Contributor

@0rzech 0rzech commented Dec 5, 2022

Fixes #1260.

@0rzech 0rzech force-pushed the test-alr-from-project-dir branch 4 times, most recently from 7f04dad to fe7f46a Compare December 5, 2022 20:05
@0rzech 0rzech force-pushed the test-alr-from-project-dir branch 2 times, most recently from a169e48 to b939210 Compare December 6, 2022 05:53
Use Polish letter 'ł' to make user input validation test
explicitly mention encoding problems when failing.
@mosteo
Copy link
Member

mosteo commented Dec 13, 2022

Looks good to me, thanks. @Fabien-Chouteau , do you think this is safe?

@Fabien-Chouteau
Copy link
Member

This can be also confusing if one wants to test another build, but it's a better default I guess.
I would prefer to have an option on the command line to specify another executable.

@mosteo
Copy link
Member

mosteo commented Dec 13, 2022

Makes sense. That can be added as a further improvement.

@mosteo mosteo merged commit a5aa0e3 into alire-project:master Dec 13, 2022
@mosteo
Copy link
Member

mosteo commented Dec 13, 2022

Thanks, @0rzech .

@0rzech 0rzech deleted the test-alr-from-project-dir branch December 13, 2022 20:15
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.

Testsuite runs any alr in $PATH
3 participants