Page MenuHomePhabricator

Change 'clang-test' to 'check-clang' on the hacking webpage
AcceptedPublic

Authored by acoomans on Jul 19 2018, 9:02 AM.

Details

Summary

The command to run tests was previously changed from 'clang-test' to 'check-clang'; with 'clang-test' remaining available as a legacy alias (see git-svn-id: https://reviews.llvm.org/rC159483 91177308-0d34-0410-b5e6-96231b3b80d8).

This commit changes the www documentation accordingly.

Diff Detail

Event Timeline

acoomans created this revision.Jul 19 2018, 9:02 AM
acoomans edited the summary of this revision. (Show Details)Jul 19 2018, 9:12 AM
JDevlieghere accepted this revision.Aug 9 2018, 8:16 AM
JDevlieghere added a subscriber: JDevlieghere.

Looks good, thanks!

PS: I don't think it hurts, but it's sufficient to add cfe-commits as a subscriber. I believe it's an automation account (so it won't be able to accept on phab) and replies via e-mail are posted under the corresponding user's account.

This revision is now accepted and ready to land.Aug 9 2018, 8:16 AM

@JDevlieghere Thanks! subscribers -> makes sense, will do

Hey Arnaud, let me know if you want me to commit this for you.