<p dir="auto"><strong>Current behavior</strong><br>
Within a form which (by error) contains two inputs fields with the same "name" attribute value (firstName is this case) Angular binds all the validations for both fields</p>
<p dir="auto"><strong>Expected behavior</strong><br>
I expected to throw an error or apply the validation only for one field</p>
<p dir="auto"><strong>Minimal reproduction of the problem with instructions</strong></p>
<ol dir="auto">
<li>Go to: <a href="https://plnkr.co/edit/iejyf2KtNhgl9PxQTbGk?p=preview" rel="nofollow">Plnkr Example Code</a></li>
<li>The first field has only "required" attribute, so enter only ONE character</li>
<li>See that "valid" property is still <em>false</em></li>
<li>Enter another two characters</li>
<li>Now "valid" is true because the second fields has also "minlength" validation</li>
</ol>
<p dir="auto"><strong>What is the motivation / use case for changing the behavior?</strong><br>
I think it's very confusing and there is almost no way to catch the problem in a big form</p>
<p dir="auto"><strong>Please tell us about your environment:</strong><br>
All</p>
<ul dir="auto">
<li>
<p dir="auto"><strong>Browser:</strong> [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]<br>
All</p>
</li>
<li>
<p dir="auto"><strong>Language:</strong> all</p>
</li>
</ul>
<p dir="auto"><strong>I'm submitting a ...</strong>  (check one with "x")</p>
<div class="snippet-clipboard-content notranslate position-relative overflow-auto" data-snippet-clipboard-copy-content="[x] bug report =&gt; search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request =&gt; Please do not submit support request here, instead see https://github.com/angular/angular/blob/master/CONTRIBUTING.md#question"><pre class="notranslate"><code class="notranslate">[x] bug report =&gt; search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request =&gt; Please do not submit support request here, instead see https://github.com/angular/angular/blob/master/CONTRIBUTING.md#question
</code></pre></div>
<p dir="auto"><strong>Current behavior</strong><br>
With either line comments or block comments, <code class="notranslate">ngc</code> includes <code class="notranslate">@imports</code> that are commented out in <code class="notranslate">.scss</code> files</p>
<p dir="auto"><strong>Expected behavior</strong><br>
Don't do imports if they are commented out</p>
<p dir="auto"><strong>Minimal reproduction of the problem with instructions</strong></p>
<ol dir="auto">
<li>Write a commented-out <code class="notranslate">@import</code> in e.g. an ng-cli-created <code class="notranslate">/src/styles.scss</code>, pointing to something that does not exist.</li>
<li>Run something like <code class="notranslate">ng build --prod &amp;&amp; ngc &amp;&amp; webpack</code></li>
</ol>
<p dir="auto"><code class="notranslate">ng build --prod</code> and <code class="notranslate">webpack</code> will ignore the commented-out lines, but <code class="notranslate">ngc</code> will follow the imports and err.</p>
<p dir="auto"><strong>What is the motivation / use case for changing the behavior?</strong><br>
Behave like the other sass build tools which ignore commented-out lines.</p>
<p dir="auto"><strong>Please tell us about your environment:</strong></p>
<blockquote>
<p dir="auto">@angular/cli: 1.1.0-beta.0<br>
node: 7.10.0<br>
os: win32 x64<br>
@angular/animations: 4.1.3<br>
@angular/common: 4.1.3<br>
@angular/compiler: 4.1.3<br>
@angular/core: 4.1.3<br>
@angular/forms: 4.1.3<br>
@angular/http: 4.1.3<br>
@angular/platform-browser: 4.1.3<br>
@angular/platform-browser-dynamic: 4.1.3<br>
@angular/platform-server: 4.1.3<br>
@angular/router: 4.1.3<br>
@angular/cli: 1.1.0-beta.0<br>
@angular/compiler-cli: 4.1.3</p>
</blockquote>