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

feat(env-collector): a few improvements for using in stats #1048

Merged
merged 14 commits into from
Sep 10, 2024

Conversation

bragov4ik
Copy link
Contributor

@bragov4ik bragov4ik commented Sep 2, 2024

  • configurable anchor postfix
  • more flexible filtering
  • keep user-defined order in the table
  • ignore non-ascii symbols in env names (to allow custom formatting with zero-width spaces, for example)
  • more compact "Required" column
  • more general allowed format for "default value" column

@bragov4ik bragov4ik force-pushed the bragov4ik/env-collector-configure-anchors branch 2 times, most recently from a4ff7c3 to add668d Compare September 3, 2024 09:40
Base automatically changed from bragov4ik/env-collector-test-case to ll/env-collector September 3, 2024 09:45
@bragov4ik bragov4ik force-pushed the bragov4ik/env-collector-configure-anchors branch from add668d to 9c732d7 Compare September 3, 2024 09:47
@bragov4ik bragov4ik changed the title feat(libs): configurable anchor postfix for env-collector feat(libs): improvements for env-collector Sep 6, 2024
@bragov4ik bragov4ik marked this pull request as ready for review September 6, 2024 09:46
Base automatically changed from ll/env-collector to main September 9, 2024 16:23
@bragov4ik bragov4ik changed the title feat(libs): improvements for env-collector feat(env-collector): a few improvements for using in stats Sep 10, 2024
@bragov4ik bragov4ik force-pushed the bragov4ik/env-collector-configure-anchors branch from 1580ed1 to 16f2ebd Compare September 10, 2024 09:06
@bragov4ik bragov4ik merged commit 36ddd8f into main Sep 10, 2024
1 of 2 checks passed
@bragov4ik bragov4ik deleted the bragov4ik/env-collector-configure-anchors branch September 10, 2024 09:22
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