Skip to content

Test alr from project dir#1262

Merged
mosteo merged 3 commits into
alire-project:masterfrom
0rzech:test-alr-from-project-dir
Dec 13, 2022
Merged

Test alr from project dir#1262
mosteo merged 3 commits into
alire-project:masterfrom
0rzech:test-alr-from-project-dir

Conversation

@0rzech

@0rzech 0rzech commented Dec 5, 2022

Copy link
Copy Markdown
Contributor

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.
@0rzech 0rzech force-pushed the test-alr-from-project-dir branch from b939210 to 8296af5 Compare December 6, 2022 05:59
@mosteo

mosteo commented Dec 13, 2022

Copy link
Copy Markdown
Member

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

@Fabien-Chouteau

Copy link
Copy Markdown
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

mosteo commented Dec 13, 2022

Copy link
Copy Markdown
Member

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

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

mosteo commented Dec 13, 2022

Copy link
Copy Markdown
Member

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