Skip to content

refactor: remove extra closing tags#30

Merged
felix-schultz merged 2 commits intomainfrom
deepsource-autofix-706b48d5
Mar 8, 2025
Merged

refactor: remove extra closing tags#30
felix-schultz merged 2 commits intomainfrom
deepsource-autofix-706b48d5

Conversation

@deepsource-autofix
Copy link
Copy Markdown
Contributor

Components without children can be self-closed to avoid the unnecessary extra closing tag. In JSX, closing tags are required when the component has children example <MyComponent>...</MyComponent> and if there are no child component between these tags, then this component can be self closed using <MyComponent />. It is recommended as it improves readability, and it is more compact to use self-closing for these types of components.

Components without children can be self-closed to avoid the unnecessary extra closing tag. In JSX, closing tags are required when the component has children example `<MyComponent>...</MyComponent>` and if there are no child component between these tags, then this component can be self closed using `<MyComponent />`. It is recommended as it improves readability, and it is more compact to use self-closing for these types of components.
This commit fixes the style issues introduced in 81dd2e3 according to the output
from Prettier, Rustfmt and StandardJS.

Details: #30
@deepsource-io
Copy link
Copy Markdown

deepsource-io bot commented Mar 8, 2025

Here's the code health analysis summary for commits d17d84a..2343248. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Docker LogoDocker✅ SuccessView Check ↗
DeepSource Rust LogoRust✅ SuccessView Check ↗
DeepSource JavaScript LogoJavaScript❌ Failure
❗ 70 occurences introduced
🎯 76 occurences resolved
View Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@felix-schultz felix-schultz merged commit 05fbcd0 into main Mar 8, 2025
7 of 10 checks passed
@felix-schultz felix-schultz deleted the deepsource-autofix-706b48d5 branch March 8, 2025 16:45
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