<div dir="auto"><div>My preference would be to put an /* eslint-disable */ at the top of the relevant files, instead. It keeps the linter green, and is a clue to future readers of the code "look out, there may be some unusual stuff in here, lint-wise"</div><div dir="auto"><br></div><div dir="auto">I worry that if we get used to the linter failing, we will stop paying attention to it.</div><div dir="auto"><br></div><div dir="auto">My $0.02, with appreciation to the release team.<br><br><div class="gmail_quote" dir="auto"><div dir="ltr" class="gmail_attr">On Fri, May 10, 2024, 8:57 AM Mike Rylander via Evergreen-dev <<a href="mailto:evergreen-dev@list.evergreen-ils.org">evergreen-dev@list.evergreen-ils.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div>+1 from me, for sure.</div><div><br></div><div data-smartmail="gmail_signature">Regards,<br><br>Mike Rylander<br><br>--   Sent from my phone, please pardon my thumbs.<br> | Research and Development Manager<br> | Equinox Open Library Initiative<br> | phone:  1-877-OPEN-ILS (673-6457)<br> | email:  miker@EquinoxInitiative.org<br> | web:  <a href="http://EquinoxInitiative.org" target="_blank" rel="noreferrer">http://EquinoxInitiative.org</a></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, May 10, 2024, 11:49 AM Bill Erickson via Evergreen-dev <<a href="mailto:evergreen-dev@list.evergreen-ils.org" target="_blank" rel="noreferrer">evergreen-dev@list.evergreen-ils.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi All,<br><div><br></div><div>We're bumping against a lot of Angular lint errors on pending feature branches as a result of relatively recent linting rules, some of which will require significant refactoring to resolve.  </div><div><br></div><div>Note that 'ng lint --fix' helps, but it only fixes certain classes of errors, mostly cosmetic.</div><div><br></div><div>These feature branches have been tested in the wild and are generally ready to go.  Efforts have been made to resolve the lint issues, but some fixes, like batch changing "==" to "===" can cause problems, since the original code was not created and tested with those assumptions in place.  Other more complicated issues could take days or longer to fix and re-test.</div><div><br></div><div>I propose we relax our requirement that merged code be fully cleared of lint errors at merge time until we get through this phase of feature merging.</div><div><br></div><div>Thoughts?</div><div><br></div><div>Thanks,</div><div><br></div><div>-b</div><div><br></div><div><br></div><div> </div></div>
_______________________________________________<br>
Evergreen-dev mailing list<br>
<a href="mailto:Evergreen-dev@list.evergreen-ils.org" rel="noreferrer noreferrer" target="_blank">Evergreen-dev@list.evergreen-ils.org</a><br>
<a href="http://list.evergreen-ils.org/cgi-bin/mailman/listinfo/evergreen-dev" rel="noreferrer noreferrer noreferrer" target="_blank">http://list.evergreen-ils.org/cgi-bin/mailman/listinfo/evergreen-dev</a><br>
</blockquote></div>
_______________________________________________<br>
Evergreen-dev mailing list<br>
<a href="mailto:Evergreen-dev@list.evergreen-ils.org" target="_blank" rel="noreferrer">Evergreen-dev@list.evergreen-ils.org</a><br>
<a href="http://list.evergreen-ils.org/cgi-bin/mailman/listinfo/evergreen-dev" rel="noreferrer noreferrer" target="_blank">http://list.evergreen-ils.org/cgi-bin/mailman/listinfo/evergreen-dev</a><br>
</blockquote></div></div></div>