-
Notifications
You must be signed in to change notification settings - Fork 32
feat: Change for testing CI #3903
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
base: main
Are you sure you want to change the base?
Conversation
Bundle ReportChanges will decrease total bundle size by 2.56kB (-0.02%) ⬇️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: gazebo-production-esmAssets Changed:
view changes for bundle: gazebo-production-systemAssets Changed:
|
Bundle ReportBundle size has no change ✅ |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3903 +/- ##
=======================================
Coverage 98.72% 98.72%
=======================================
Files 827 827
Lines 15040 15040
Branches 4298 4306 +8
=======================================
Hits 14848 14848
Misses 185 185
Partials 7 7
Continue to review full report in Codecov by Sentry.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅ ✅ All tests successful. No failed tests found. @@ Coverage Diff @@
## main #3903 +/- ##
=======================================
Coverage 98.72% 98.72%
=======================================
Files 827 827
Lines 15040 15040
Branches 4306 4306
=======================================
Hits 14848 14848
Misses 185 185
Partials 7 7
Continue to review full report in Codecov by Sentry.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. @@ Coverage Diff @@
## main #3903 +/- ##
=======================================
Coverage 98.72% 98.72%
=======================================
Files 827 827
Lines 15040 15040
Branches 4306 4298 -8
=======================================
Hits 14848 14848
Misses 185 185
Partials 7 7
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. @@ Coverage Diff @@
## main #3903 +/- ##
=======================================
Coverage 98.72% 98.72%
=======================================
Files 827 827
Lines 15040 15040
Branches 4306 4306
=======================================
Hits 14848 14848
Misses 185 185
Partials 7 7
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
✅ Sentry found no issues in your recent changes ✅ |
✅ Deploy preview for gazebo ready!Previews expire after 1 month automatically.
|
@sentry review |
On it! We are reviewing the PR and will provide feedback shortly. |
PR DescriptionThis pull request removes an unnecessary and potentially confusing comment from the Click to see moreKey Technical ChangesThe only technical change is the removal of the single-line comment Architecture DecisionsNo architectural decisions are involved in this change. It's a simple cleanup of an existing component. Dependencies and InteractionsThis change does not introduce any new dependencies or affect interactions with other parts of the system. It's a localized modification within a single component. Risk ConsiderationsThere are no foreseeable risks associated with this change. The removal of the comment does not alter the functionality of the component. Notable Implementation DetailsThe change is a straightforward deletion of a comment. No special attention is required during review. |
STOP: This is just for testing CI workflow.