Skip to content

Update Main.tsx - #90

Open
NejlaSetkic wants to merge 1 commit into
mainfrom
NejlaSetkic-patch-96
Open

Update Main.tsx#90
NejlaSetkic wants to merge 1 commit into
mainfrom
NejlaSetkic-patch-96

Conversation

@NejlaSetkic

Copy link
Copy Markdown
Owner

No description provided.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Quality Gates: OK

  • Declining Code Health: 0 findings(s) 🚩
  • Improving Code Health: 1 findings(s) ✅
  • Affected Hotspots: 1 files(s) 🔥

Recommended Review Level: Inconclusive -- Not enough commits to recommend a review strategy. The recommendation will be enabled automatically once you have more development activity.
View detailed results in CodeScene

✅ Positive Code Health Impact (highest to lowest):

Comment thread src/Main.tsx
import {Website} from './Website';
import {WebTechnologies} from './WebTechnologies';

export const Main: React.FC = () => {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ No longer an issue: Large Method
Main:React.FC is no longer above the threshold for lines of code

Why does this problem occur?

Large functions with many lines of code are generally harder to understand and lower the code health. Avoid adding more lines to this function. Read more.

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