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

ci: Use a local httpbin instance for tests #166

Closed
wants to merge 4 commits into from
Closed

Conversation

janbuchar
Copy link
Collaborator

@janbuchar janbuchar requested a review from vdusek May 31, 2024 16:39
@github-actions github-actions bot added this to the 90th sprint - Tooling team milestone May 31, 2024
@github-actions github-actions bot added t-tooling Issues with this label are in the ownership of the tooling team. tested Temporary label used only programatically for some analytics. labels May 31, 2024
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

⚠️ Pull Request Tookit has failed!

None of the pull request and linked issue has estimate

Copy link
Collaborator

@vdusek vdusek left a comment

Choose a reason for hiding this comment

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

LGTM, nice solution 👍

@janbuchar
Copy link
Collaborator Author

@vdusek not so fast, it looks like the changes did not actually do anything 😄 I'll look into it Monday.

@janbuchar janbuchar closed this Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t-tooling Issues with this label are in the ownership of the tooling team. tested Temporary label used only programatically for some analytics.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Run httpbin locally instead of using httpbin.org
2 participants