-
Notifications
You must be signed in to change notification settings - Fork 1
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
Mrc 6043 tidy metadata page #138
Conversation
Previously, I saw: 'Warning: React version not specified in eslint-plugin-react settings. See https://github.com/jsx-eslint/eslint-plugin-react\#configuration'
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #138 +/- ##
=======================================
Coverage 97.22% 97.22%
=======================================
Files 131 131
Lines 1225 1227 +2
Branches 339 341 +2
=======================================
+ Hits 1191 1193 +2
Misses 33 33
Partials 1 1 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
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.
LGTM just a question left about bang operator
<h3 className="text-lg font-bold tracking-tight">Timings</h3> | ||
</span> | ||
} | ||
<ul className="ps-1 flex gap-10 !mt-3"> |
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.
whats the ! for? is it needed?
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.
Workaround pushed
In this PR, I capitalized some lower case placeholder text on the home page, configured eslint so that it would run on my machine, and, primarily, introduced some organisation onto the metadata page.
https://mrc-ide.myjetbrains.com/youtrack/issue/mrc-6043/Make-git-execution-time-metadata-presentation-more-structured
Before
After