Fix portability for expr(1) usage #1
Merged
SlavekB
merged 1 commits from feat/fix-expr-portability
into master
5 years ago
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'feat/fix-expr-portability'
Deleting a branch is permanent. It CANNOT be undone. Continue?
"match" operand for expr(1) is GNU extension.
Change to use the standard way to match regex with expr(1).
Everything looks good.
WIP: Fix portability for expr(1) usageto Fix portability for expr(1) usage 5 years agoThanks Obata-san.
Just as a reminder, there is no need to add WIP in the title of the PR.
A PR marked as WIP means you intend to do more work on it and should not be merged yet.
I read the TGW guide again and there is no mentioning of WIP, unless I totally missed it.
@obache, you can use a different way. Instead of WIP to the PR name, you can set label PR/rfc. By doing so, you acknowledge that your work is completed and you want us to review the proposed PR.
Have you ever created PR here?
below the title input area:
"Start the title with WIP: to prevent the pull request from being merged accidentally."
Yes, but it does not make sense to keep the WIP tag when you really want to merge 😺
As we said – WIP means that you're still working on PR == is not ready for merge. If PR is ready to merge, the WIP tag is confusing.
8b1ffe85f3
.