-
Notifications
You must be signed in to change notification settings - Fork 197
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
Dependency details do not show test-suite dependencies #809
Comments
#572 is also related. |
To extend my former criticism about this feature: we can add this to the details UI, and reduce the clutter effect by means of making the less-relevance of test-suite dependencies apparent by means of appropriate CSS styling so it is shown in a less salient way. Or alternatively, by having an expandable collapsed section for the non-primary components. |
On Wed, 23 Jan 2019, Herbert Valerio Riedel wrote:
To extend my former criticism about this feature: we can add this to the
details UI, and reduce the clutter effect by means of making the
less-relevance of test-suite dependencies by means of appropriate CSS
styling so it is shown in a less salient way.
If the users clicks on details he wants to see the details, won't he?
If I don't see test-suite dependencies in the details I will assume that
there is no test-suite, at all.
|
test-suites is a component type, we don't show many. Currently There should be clear sections:
I think if there are section headers, we don't need to collapse test-suites or benchmarks. (I also hope we get per-component solving soon enough, so executables become non-primary components as well). As a (somewhat mandatory) bonus, dependencies to internal libraries should be visually distinguishable as well. |
Bump & +1 to @phadej 's "somewhat mandatory bonus": it would be nice if we didn't have internal libs listed in the dependency list; they are a broken link, and accidentally serve to hide what dependencies actually go into the lib. |
Currently, in this example the dependencies of the test-suite are not shown:
https://hackage.haskell.org/package/lapack-0.2.1/dependencies
Maybe related to #789.
The text was updated successfully, but these errors were encountered: