Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
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
Feature/intervention #2343
Feature/intervention #2343
Changes from 1 commit
23261b4
140cb0d
a3e6a30
067d46d
3afbe9e
3e17a09
d861be7
b4f31fc
411c121
2d6de25
8b9ccf9
1659928
282b8e4
96a027b
43d3856
9f9021c
8bfe467
7829842
b3cfe1b
9c4e9b5
bcabeb4
578a4c2
9a88fe8
2d82886
57d4527
59b6165
0b86b1f
588bfc2
171adf8
b22fc3a
a7555c3
4c6de8d
1ac22f6
6ea3699
7c1e9e9
6bf4248
548994b
555e3ec
d7dc806
36de55d
72797b1
98c4379
0225fc9
6aa75f4
b84ea21
364dae1
b80d951
78492b4
9f802c2
c1d748f
10096f6
b51cf14
a35837b
01c9cfd
9214c2b
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Avoid
any
forspecies.treeCount
.Please define a specific numeric type (e.g.,
number
) fortreeCount
to ensure type safety.📝 Committable suggestion
🧰 Tools
🪛 GitHub Check: CodeFactor
[warning] 55-55: src/features/projectsV2/ProjectDetails/components/OtherInterventionInfo.tsx#L55
Unexpected any. Specify a different type. (@typescript-eslint/no-explicit-any)
[warning] 55-55: src/features/projectsV2/ProjectDetails/components/OtherInterventionInfo.tsx#L55
Unexpected any. Specify a different type. (@typescript-eslint/no-explicit-any)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Refactor
renderMetaData
for better maintainability.The function has high complexity with multiple nested conditions. Consider breaking it down into smaller, focused functions:
📝 Committable suggestion
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Avoid using array index as key prop.
Using array indices as keys can lead to rendering issues when the list items are reordered or modified. Consider using a more stable unique identifier.
📝 Committable suggestion