Skip to content

Conversation

@ChrisRackauckas
Copy link
Contributor

Fixes #27

Copy link
Contributor

@JoshuaLampert JoshuaLampert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. This also needs to be adjusted in the README.

README.md Outdated

- **`deps`**: Minimize only your direct dependencies (recommended for most packages)
- **`alldeps`**: Minimize direct dependencies and weak dependencies
- **`all`**: Minimize all packages (may test issues in transitive dependencies)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- **`weakdeps`**: Minimize only weak dependencies

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should replace the line with with the all mode. Seems like the new GitHub review is still buggy with review comments at locations not touched by the PR.

@ChrisRackauckas
Copy link
Contributor Author

I think that got them all now.

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.

mode: 'all' does not work (v2)

2 participants