... but should optimally produce:
Note: in the ideal world we should generate the following, although this seems too difficult - just a suggestion for future:
I’ve updated the description since it was misleading.
If you uncomment the directives, could you still produce the following? https://take.ms/VwFra
I definitely think we should uncomment directives.
If you can make this change and post a sample “before“ and “after“, that would be awesome.
Let’s still have a short call tomorrow to summarize what remains.
We discussed it and decided to eliminate commenting for converted lines.
I made two revisions both without commenting. The first checks whether the directives are insideCompoundStatement and if not, doesn’t try to indent it. The second tries to indent all the converted outcome(and even swaps negate branches).
Put this in review if all done here.
Ok, I assumed last call, we decided to follow the second version(last commit on Feature/AvailabilitySupport).
Great. Just merged Feature/AvailabilitySupportto Develop.
Once you are done with that branch you may delete it.
The error rate (with the set of our test projects) looks good in both of your branches, so we are fine to release those changes.