Skip to content

Fix/material expected#96

Open
himanshuc3 wants to merge 2 commits intodevelfrom
fix/material_expected
Open

Fix/material expected#96
himanshuc3 wants to merge 2 commits intodevelfrom
fix/material_expected

Conversation

@himanshuc3
Copy link
Copy Markdown
Contributor

All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • [ x Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Feature Submissions:

  1. Does your submission pass tests?
  2. Have you lint your code locally prior to submission?

Changes

  • Add Assert function for Material that has color as settable.
  • If Scale is set as 0, change to a default of 0.000001 to stop warnings by THREE.js.

@lgtm-com
Copy link
Copy Markdown

lgtm-com bot commented Oct 11, 2020

This pull request introduces 1 alert when merging fe7cfa7 into 28c213a - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

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.

1 participant