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

Add WARNS check. #81

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add WARNS check. #81

wants to merge 2 commits into from

Commits on Aug 28, 2023

  1. Add WARNS check.

    The SIGNALS check can correctly detect whether a block of code signals
    a warning, but it also aborts the code block's execution. That's
    appropriate for checking for an error, but not for a warning. This new
    check confirms that a warning of a particular class is raised, a la
    SIGNALS, but also allows the block of code in its scope to complete,
    which may be more natural for some cases.
    rpgoldman committed Aug 28, 2023
    Configuration menu
    Copy the full SHA
    d6b750d View commit details
    Browse the repository at this point in the history

Commits on Sep 2, 2023

  1. Replace new WARNS macro with SIGNALS rewrite.

    SIGNALS now captures the return value of the signaling block, EXCEPT
    when the condition signaled is an error, in which case there may not
    be a value to return.
    
    Changes in tests.lisp illustrate the trade-offs.
    rpgoldman committed Sep 2, 2023
    Configuration menu
    Copy the full SHA
    1cb2463 View commit details
    Browse the repository at this point in the history