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.

Sign up for GitHub

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

Jump to bottom

promote scripting from rust scratch tls #377

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

promote scripting from rust scratch tls #377

wants to merge 2 commits into from
+4,229 −823

Conversation

nazmulidris
Copy link
Member

  • [core][tui] Clean up emoji use in logs, enhance tracing support
  • [all] Promote scripting.rs from rust_scratch/tls

@nazmulidris nazmulidris force-pushed the promote-scripting-from-rust_scratch_tls branch 29 times, most recently from 7daff4d to 5f3546f Compare December 20, 2024 18:59
@nazmulidris nazmulidris force-pushed the promote-scripting-from-rust_scratch_tls branch 21 times, most recently from 7819e24 to 8e080e8 Compare December 23, 2024 15:26
@nazmulidris
[all][script][log] Promote scripting.rs from rust_scratch/tls
4cb6759 
Create 2 new crates:
- script
- log
  - support UnicodeString for output
  - update all calls to tracing::*() to work seamlessly with CustomEventFormatter
    across the entire r3bl codebase (especially r3bl_tui examples)
  - add of the `options: impl Into<TracingConfig>` pattern for configuring the
    log functionality. this is ergonomic for simple cases (for the caller) and
    provides sophisticated configuration possibilities due to its composability.

Reorganize all the existing crates as well, to ensure that
known duplicated functions are removed.

Add new glyphs that are unicode character based, not emojis,
so that the fg, bg color and attributes are applied in output.
Emojis come with their own colors, so the fg and bg colors don't
get applied.

Use Vec::with_capacity wherever possible to prevent having reallocations
which are expensive, and create a global size limit for the all the r3bl
crates.
@nazmulidris nazmulidris force-pushed the promote-scripting-from-rust_scratch_tls branch from 8e080e8 to 4cb6759 Compare December 23, 2024 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@nazmulidris