AdaptiveCpp icon indicating copy to clipboard operation
AdaptiveCpp copied to clipboard

NFC: Remove trailing whitespace

Open hdelan opened this issue 1 year ago • 4 comments

Non functional change. Removing trailing whitespace.

hdelan avatar Jan 30 '24 17:01 hdelan

Wow, thank you! This PR touches pretty much every file. Would it be possible to split it up into smaller PRs for the different components of the project? Otherwise handling all the merge conflicts might be difficult.

illuhad avatar Jan 31 '24 14:01 illuhad

Yep sure no problem. Let me know how I should split this up into smaller PRs

hdelan avatar Jan 31 '24 14:01 hdelan

Perhaps one simple heuristic might be by subdirectory? Given that all of this is NFC, we can group the changes arbitrarily, so I don't have a strong opinion.

illuhad avatar Jan 31 '24 15:01 illuhad

OK sure so is it enough to subdivide into include, src and test?

hdelan avatar Jan 31 '24 15:01 hdelan