Skip to content

better rhai logging#1521

Merged
garypen merged 8 commits intomainfrom
garypen/better-rhai-logging
Aug 17, 2022
Merged

better rhai logging#1521
garypen merged 8 commits intomainfrom
garypen/better-rhai-logging

Conversation

@garypen
Copy link
Copy Markdown
Contributor

@garypen garypen commented Aug 17, 2022

A small ergonomic improvement for rhai logging which makes the the various log functions accept a Dynamic rather than requiring a &str. This isn't a breaking change, because a rhai Dynamic can represent any valid rhai type.

In a more ideal world, it would be easy to register variadic functions to the rhai engine, but this is at least a small improvement on the existing situation.

Gary Pennington added 6 commits August 16, 2022 13:17
To fix versioning of crates...
or things won't build too well.
Many changes in API for testing, so make sure to keep up with them.
The small logging improvements are worth releasing now. The span stuff
is not quite fully baked and needs a bit more thought. I'll think about
it for a later PR.
@garypen garypen self-assigned this Aug 17, 2022
@github-actions

This comment has been minimized.

to describe the new feature
Copy link
Copy Markdown
Contributor

@Geal Geal left a comment

Choose a reason for hiding this comment

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

nice :)

@garypen garypen merged commit c8de566 into main Aug 17, 2022
@garypen garypen deleted the garypen/better-rhai-logging branch August 17, 2022 10:02
@Geal Geal mentioned this pull request Aug 22, 2022
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