asynchronous-injection icon indicating copy to clipboard operation
asynchronous-injection copied to clipboard

Remove unused directives. Fix typo in a comment.

Open Crokus opened this issue 7 years ago • 2 comments

Crokus avatar Feb 19 '19 13:02 Crokus

Thank you for your interest in improving this code base.

I'd prefer not removing 'standard' namespaces, because, as @moodmosaic has already pointed out, you always end up having to add them again.

You could argue that this is just sample code, so maintainability probably doesn't matter as much as it would for a 'real' code base, but if that's the argument, then why bother removing any of the using directives in the first place?

ploeh avatar Feb 19 '19 18:02 ploeh

Originally I wanted to change the response from the Controller but looks like asking first was a good idea :)

Crokus avatar Feb 19 '19 19:02 Crokus