Skip to content

Conversation

@ChetanFTW
Copy link
Member

Notes for Reviewers

  • Live Coordinates help us to see the points while using ShapeBuilder
  • Removed unwanted Css

Signed commits

  • Yes, I signed my commits.

@Namanv0509
Copy link
Contributor

@ChetanFTW can you fix the lint issue here.

@ChetanFTW
Copy link
Member Author

so, what just happen is the workflow is not work due to some packages -- therefore i resolved in this PR, whenenever this PR merge will create another PR to resolve this one

@M-DEV-1
Copy link
Member

M-DEV-1 commented Nov 4, 2025

@ChetanFTW, check the workflow errors.

/home/runner/work/shape-builder/shape-builder/site/src/components/Navigation/index.js
  12:43  warning  'showSignUpButton' is defined but never used  no-unused-vars

/home/runner/work/shape-builder/shape-builder/site/src/components/ShapeBuilder/index.js
  163:1  error  Expected indentation of 8 spaces but found 2   indent
  164:1  error  Expected indentation of 10 spaces but found 4  indent
  165:1  error  Expected indentation of 10 spaces but found 4  indent
  166:1  error  Expected indentation of 10 spaces but found 4  indent
  167:1  error  Expected indentation of 10 spaces but found 4  indent
  168:1  error  Expected indentation of 8 spaces but found 2   indent

We could modify the file to increase/decrease the strictness of the checks.

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.

3 participants