search

ruricolist / serapeum

Utilities beyond Alexandria
MIT License
420 stars 41 forks source link

Remove AGNOSTIC-LIZARD #111

Closed phoe closed 2 years ago

phoe commented 2 years ago

Fixes #110

phoe commented 2 years ago

https://app.travis-ci.com/github/phoe-trash/serapeum/jobs/557125258

TEST-WITH-BOOLEAN-EXPANSION fails on CCL because CCL's macroexpander seems to strip away all MACROLET forms from the resulting expansion. (I guess it's allowed to do that, since they are no longer necessary in the expansion.)

What do we do? Does it make sense to remove that test as well and only perform black-box testing of WITH-BOOLEAN?

ruricolist commented 2 years ago

Note that trivial-macroexpand-all is already a dependency of Serapeum. But I would just drop that test. Depending on the precise details of the full macroexpansion strikes me as brittle anyway.

phoe commented 2 years ago

Dropped the test, waiting for CI.

phoe commented 2 years ago

CI tells me it's fixed, should be ready for merging.

Apologies for the accidental supply chain attack, I'll be more careful in the future.