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

add: functional tests #5

Merged
merged 2 commits into from
Aug 17, 2023
Merged

add: functional tests #5

merged 2 commits into from
Aug 17, 2023

Conversation

jimf5
Copy link
Collaborator

@jimf5 jimf5 commented Jun 14, 2023 •
edited
Loading

Proposed changes

Added functional tests.

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING document
  • If applicable, I have added tests that prove my fix is effective or that my feature works
  • If applicable, I have checked that any relevant tests pass after adding my changes
  • I have updated any relevant documentation (README.md and CHANGELOG.md)

@jimf5 jimf5 changed the title add: functional tests for HTTP and HTTP/2 add: functional tests Jul 12, 2023
@jimf5 jimf5 force-pushed the add_tests branch 2 times, most recently from 7f05dfc to 0569a2b Compare July 12, 2023 19:34
@jimf5 jimf5 force-pushed the add_tests branch 18 times, most recently from 264b5c1 to 222734a Compare August 15, 2023 23:18
route443
route443 approved these changes Aug 17, 2023
View reviewed changes
Copy link
Collaborator

@route443 route443 left a comment

Choose a reason for hiding this comment

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

LGTM

@jimf5 jimf5 merged commit 3d528bb into main Aug 17, 2023
2 checks passed
@jimf5 jimf5 deleted the add_tests branch December 6, 2023 04:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@route443 route443 route443 approved these changes

@dplotnikov-f5 dplotnikov-f5 Awaiting requested review from dplotnikov-f5

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

Successfully merging this pull request may close these issues.
2 participants
@jimf5 @route443