Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Avoid powerapps checker strict equality issues #4

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

filcole
Copy link

@filcole filcole commented Jul 10, 2021

Webpack production mode can optimise === into ==, which causes the power apps solution checker to complain. This PR disables the optimisation for this specific area.

I'm seeing between 0.3% and 1.8% increase in bundle size due to turning off this optimisation.

Alternatives are:

  • Exclude the specific rule when running solution checker - which is only possible when running solution checker via Powershell, and not (yet) possible with the standard Power Platform Build Tools Azure DevOps tasks or maker portal
  • Ignore errors :(

@scottdurow
Copy link
Owner

Thanks @filcole - that checker rule is a mystery to me - I really don't think it should be there at all - just like the .top rule - it causes many false positives.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants