Skip to content

chore(statevars): remove unused useEffect from CreateStateOperation#342

Merged
leowla merged 1 commit intomasterfrom
chore/unused-import
Apr 5, 2026
Merged

chore(statevars): remove unused useEffect from CreateStateOperation#342
leowla merged 1 commit intomasterfrom
chore/unused-import

Conversation

@leowla
Copy link
Copy Markdown
Member

@leowla leowla commented Apr 2, 2026

Describe the issue

Failing frontend lint check. Causing red CI.

Describe the solution

Remove unused useEffect import

Risk

None

Definition of Done

  • Code peer-reviewed
    - [ ] Wiki Documentation is written and up to date
    - [ ] Unit tests written and passing
    - [ ] Integration tests written and passing
  • Continuous Integration build passing
    - [ ] Acceptance criteria met
    - [ ] Deployed to production environment

Reviewed By

Who reviewed your PR - for commit history once merged

@leowla leowla self-assigned this Apr 2, 2026
@leowla leowla added the frontend label Apr 2, 2026
@leowla leowla requested a review from Copilot April 2, 2026 04:27
Copy link
Copy Markdown
Contributor

@harbassan harbassan left a comment

Choose a reason for hiding this comment

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

awesome, cheers.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes an unused React hook import in CreateStateOperation to resolve a failing frontend lint check and restore green CI.

Changes:

  • Removed the unused useEffect import from CreateStateOperation.jsx.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@leowla leowla requested a review from hazikchaudhry April 3, 2026 01:00
Copy link
Copy Markdown
Contributor

@hazikchaudhry hazikchaudhry 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 👍🏽

@leowla leowla merged commit c8995e6 into master Apr 5, 2026
11 checks passed
@leowla leowla deleted the chore/unused-import branch April 5, 2026 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants