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

timeline deserializer #7977

Merged
merged 6 commits into from
Sep 19, 2024
Merged

timeline deserializer #7977

merged 6 commits into from
Sep 19, 2024

Conversation

jackkav
Copy link
Contributor

@jackkav jackkav commented Sep 19, 2024 •
edited
Loading

taming CLI verbose output

  • use a common tested deserializer
  • fix the spacing of the output

@jackkav jackkav marked this pull request as ready for review September 19, 2024 13:47
@jackkav jackkav requested a review from a team September 19, 2024 13:51
filfreire
filfreire previously approved these changes Sep 19, 2024
View reviewed changes
gatzjames
gatzjames previously approved these changes Sep 19, 2024
View reviewed changes
@jackkav jackkav dismissed stale reviews from gatzjames and filfreire via a9b31b0 September 19, 2024 14:35
@jackkav jackkav force-pushed the improve-cli-timeline-deserializer branch from 55491c1 to a9b31b0 Compare September 19, 2024 14:35
@jackkav jackkav enabled auto-merge (squash) September 19, 2024 14:35
@jackkav jackkav merged commit 76c0130 into Kong:develop Sep 19, 2024
8 checks passed
@jackkav jackkav deleted the improve-cli-timeline-deserializer branch September 19, 2024 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@filfreire filfreire filfreire approved these changes

@gatzjames gatzjames gatzjames left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jackkav @filfreire @gatzjames