Skip to content

Conversation

@CDeltakai
Copy link
Contributor

@CDeltakai CDeltakai commented Apr 22, 2025

Description

This PR integrates the new js-lint package and removes old linting elements

Issues Fixed

Tasks

  • 1. Integrate js-lint and ensure it works as expected.
  • 2. Remove old linting elements as they are no longer needed.

Final checklist

  • Domain specific tests
  • Full tests
  • Updated inline-comment documentation
  • Lint fixed
  • Squash and rebased
  • Sanity check the final build

@CDeltakai CDeltakai self-assigned this Apr 22, 2025
@CDeltakai CDeltakai requested a review from aryanjassal May 13, 2025 07:53
@CDeltakai
Copy link
Contributor Author

@aryanjassal Could you just give it a quick check for the latest commit? I just removed the await from some of the for loops since the linter was saying that it was incorrect because the iterables were synchronous. Just want to make sure this doesn't break functionality before merging.

@CDeltakai CDeltakai merged commit 3b02c97 into staging May 14, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

js-rpc: Integrate js-lint

2 participants