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.

Sign up for GitHub

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

Jump to bottom

Code Quality Improved: Removed the unused variables #4495

Merged
merged 1 commit into from
Jul 9, 2023
Merged

Code Quality Improved: Removed the unused variables #4495

merged 1 commit into from
Jul 9, 2023

Conversation

RakshitKumar04
Copy link
Contributor

This PR fixes #4492

Notes for Reviewers

  • Corrected the 6 files which showed warnings, but could not find the indentation error as mentioned in the screenshot.
  • Orgs/layer5/src/sections/Community/Member-single/index.js
    267:1 already has 18 spaces, in spite of 20.

Signed commits

  • Yes, I signed my commits.

@github-actions github-actions bot added area/blog New posts or new blog functionality project/kanvas labels Jul 6, 2023
@l5io
Copy link
Contributor

l5io commented Jul 6, 2023

🚀 Preview for commit 454c164 at: https://64a723e76555d177f26c476d--layer5.netlify.app

@Mohith234
Copy link
Member

@RakshitKumar04 Don't raise any PRs unless you get assigned. Please follow the contribution guidelines.

@RakshitKumar04
Removed the unused variables
87452f5 
Signed-off-by: Rakshit Sharma <kumarrakshit0402@gmail.com>
@l5io
Copy link
Contributor

l5io commented Jul 7, 2023

🚀 Preview for commit 87452f5 at: https://64a83b5c48013d00736d48ce--layer5.netlify.app

leecalcote
leecalcote approved these changes Jul 9, 2023
View reviewed changes
Copy link
Member

@leecalcote leecalcote left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thanks, @RakshitKumar04 👍

@leecalcote leecalcote merged commit b0d7677 into layer5io:master Jul 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leecalcote leecalcote leecalcote approved these changes

Assignees
No one assigned
Labels
area/blog New posts or new blog functionality project/kanvas
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Code Quality Improvement: Variable declared but not in use.
4 participants
@RakshitKumar04 @l5io @Mohith234 @leecalcote