Skip to content

Conversation

@Binlogo
Copy link

@Binlogo Binlogo commented Nov 3, 2025

Update README.md to fit #248 cargo_bin deprecation.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 19035595145

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 39.463%

Totals Coverage Status
Change from base Build 18990122291: 0.0%
Covered Lines: 191
Relevant Lines: 484

💛 - Coveralls

@epage
Copy link
Contributor

epage commented Nov 3, 2025

Note that in #248, I said

The problem with this is we can't compile doctests using cargo_bin!. Unsure what the right answer is.

@Binlogo
Copy link
Author

Binlogo commented Nov 4, 2025

🤔 Didn't realize it included the README as doctests as well. Perhaps temporarily ignore the README for convenience, until it's figured out. Anyway, it's an interesting issue.

Note that in #248, I said

The problem with this is we can't compile doctests using cargo_bin!. Unsure what the right answer is.

@epage
Copy link
Contributor

epage commented Nov 4, 2025

As cargo_bin_cmd! isn't the only feature shown here, I'm wanting to lean on being consistent with #248 and just leave it as-is for now.

@epage epage closed this Nov 4, 2025
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.

3 participants