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

Config updater improvements #151

Merged
merged 21 commits into from
Sep 26, 2023

Conversation

tkmcmaster
Copy link
Collaborator

@tkmcmaster tkmcmaster commented Sep 22, 2023

  • Major improvements to the pewpew-config-updater. It will now do everything it can to try and leave the old expressions in and attempt to convert them over
    • Known issues:
    • Expressions in vars will not wrap environment variables in the expected ${e:VAR}
    • vars in logs and provides will not have the prepended _v. before the var name.
  • Changed the order of limit in loggers
  • Removed cargo deny ignore now that the dependency is removed

- Can now parse epoch and encode as well as basic response.body.x
- If there was an comparison operators in an expression, it caused a constant stack climb and it repeatedly called display
- Also cleaned up some of the log messages
- Added fix for empty strings that are parsed as templates with no pieces
- Added initial code to allow parsing of Segments with rest into multiple segments
@tkmcmaster tkmcmaster self-assigned this Sep 22, 2023
@tkmcmaster tkmcmaster marked this pull request as ready for review September 26, 2023 20:04
@tkmcmaster tkmcmaster merged commit de04df8 into 0.6.0-scripting-dev Sep 26, 2023
18 checks passed
@tkmcmaster tkmcmaster deleted the config-updater-improvements branch October 2, 2023 13:33
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