Skip to content

Conversation

epage
Copy link
Collaborator

@epage epage commented Oct 16, 2025

This is part of #13

I went with ctor because

  • Divan ran into issues with linkme, see nvzqz/nvzqz/divan/pull/32divan#32
  • It doesn't involve proc-macros or build scripts

Like ctors optional proc-macro, I kept out proc-macro minimal to experiment with what it might take to use declarative attributes when they are ready.

Known issues:

  • Only () is allowed as a return type
  • No #[should_panic]
  • No #[ignore]
  • Reported order for tests is weird

@coveralls
Copy link

coveralls commented Oct 16, 2025

Pull Request Test Coverage Report for Build 18571071689

Details

  • 17 of 91 (18.68%) changed or added relevant lines in 4 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.7%) to 30.432%

Changes Missing Coverage Covered Lines Changed/Added Lines %
crates/distributed_list/src/list.rs 14 25 56.0%
crates/libtest2/src/case.rs 0 12 0.0%
crates/libtest2-proc-macro/src/lib.rs 0 51 0.0%
Totals Coverage Status
Change from base Build 18564295825: -0.7%
Covered Lines: 500
Relevant Lines: 1643

💛 - Coveralls

@epage epage merged commit ae906fd into assert-rs:main Oct 16, 2025
17 checks passed
@epage epage deleted the macro branch October 16, 2025 18:41
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