Skip to content

fix: resolves 10 of 14 SonarCloud code smells - #1337

Open
Ishita-190 wants to merge 2 commits into
asyncapi:masterfrom
Ishita-190:newer
Open

fix: resolves 10 of 14 SonarCloud code smells#1337
Ishita-190 wants to merge 2 commits into
asyncapi:masterfrom
Ishita-190:newer

Conversation

@Ishita-190

Copy link
Copy Markdown
Contributor

Description
This PR resolves multiple SonarCloud maintainability and reliability issues across the library.

  • Updated util import to node:util in jest.setup.js
  • Replaced typeof ... === 'undefined' checks with direct undefined comparisons
  • Removed the unnecessary React fragment in Extensions.tsx
  • Added type="button" attributes to schema tab buttons
  • Marked XExtension component props as readonly
  • Merged duplicate React imports in ErrorBoundary.tsx
  • Removed unused commented out CSS

Related issue(s)

SonarCloud issues

@sonarqubecloud

Copy link
Copy Markdown

@Ishita-190 Ishita-190 changed the title fix: resolves 10/14 SonarCloud code smells fix: resolves 10 of 14 SonarCloud code smells Aug 17, 2026
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