Skip to content
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

added a param whether to include reverse dependencies #26

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Zhenglei-BCS
Copy link
Collaborator

No description provided.

@Zhenglei-BCS Zhenglei-BCS marked this pull request as draft February 20, 2025 15:26
@Zhenglei-BCS
Copy link
Collaborator Author

Zhenglei-BCS commented Feb 20, 2025

  • Environment report contains too much information, or do we assume that it is always being ran in a clean environment, like a docker container where there is not much additional information to be reported?
  • for example:
    image

I feel instead of exclude some outputs, we can define the included ones.

@llrs-roche
Copy link
Collaborator

Thanks for opening the PR, I thought I had created an issue for this !!👍

While this won't display the reverse dependencies it will still show the section. I would prefer to avoid having the section completely if we don't want the reverse dependencies. We'll need to think how to do that here.


The function generating the environment output that has two parameters show to only show those written there and exclude to show everything and exclude some values. Similarly other functions have these two parameters.

For all the system settings/configs there is a separate branch on #19 to hide the content. I think there is value on having it but simply not show it unless the user wants it.

In general I think the default template can't do everything, but can show how to use the functionality on the package/template to use for a wide range of functionality, like on this PR were we will be excluding a section.

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.

2 participants