Repository navigation
Conversation
| fail-fast: false | ||
| matrix: | ||
| job: | ||
| - { features: "feat_Tier1,feat_require_unix" } |
There was a problem hiding this comment.
I might hardcoded feat_os_unix at previous PR.
There was a problem hiding this comment.
Yes, you removed feat_os_unix in #15547. I don't think that is right though as we now don't check feat_Tier1.
There was a problem hiding this comment.
@oech3 matrix build is unnecessary though, how about:
env:
FEATURES: "feat_Tier1,feat_require_unix"There was a problem hiding this comment.
It FEATURES something officially supported by cargo? If not, I'd avoid environment variables to avoid sideffects.
There was a problem hiding this comment.
It
FEATURESsomething officially supported bycargo?
No it is not. However, using a build matrix makes the job name more complex, for example:
Style and Lint (feat_Tier1,feat_require_unix)
instead of
Style and Lint
There was a problem hiding this comment.
Keep this thread and wait maintainer's review.
5241c2a to
17a0ba3
Compare
Merging this PR will not alter performance1 warning, 1 note
Comparing Footnotes
|
17a0ba3 to
b602afe
Compare
utmpx not supported for opensbd
b602afe to
e19098c
Compare
utmpx not supported for opensbd