I was going to apply this locally, but maybe it makes sense in the upstream?
Details
Details
Diff Detail
Diff Detail
- Repository
- rARC Arcanist
- Branch
- master
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 2522 Build 2526: [Placeholder Plan] Wait for 30 Seconds
Event Timeline
Comment Actions
I think this should remain an error by default -- although work in T4045 and elsewhere will continue making this less severe than it once was, I think it's still worth raising on non-changed lines in the general case. I don't think there's a super strong use case for mixing character sets in source code, and .arclint has reasonable tools for excluding small sets of, e.g., test data now.