Skip to content

Lazy execute recorded as block#51

Merged
jecisc merged 4 commits intojecisc:developmentfrom
Gabriel-Darbord:lazy-execute-recordedAs-block
Jun 19, 2025
Merged

Lazy execute recorded as block#51
jecisc merged 4 commits intojecisc:developmentfrom
Gabriel-Darbord:lazy-execute-recordedAs-block

Conversation

@Gabriel-Darbord
Copy link
Contributor

Similar to #47
Add lazy behavior to TinyLogger>>#execute:recordedAs: such that:

  • If no loggers are set, then the execute block runs immediately.
  • The second argument can be a block that will only be evaluated if necessary.

In addition:

  • Rename arg aString to aBlockOrString where applicable.
  • Add space before error message when execute:recordedAs: is interrupted.
  • Remove use of value:during: in TinyLoggerTest, since it is now done in performTest.
  • Add a test for value:during: to check that the previous logger is restored.

Copy link
Owner

@jecisc jecisc left a comment

Choose a reason for hiding this comment

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

Thank!

@jecisc jecisc merged commit 1b9d652 into jecisc:development Jun 19, 2025
10 checks passed
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