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

Fix TEST_HOST name mismatch #177

Conversation

CongLeSolutionX
Copy link
Contributor

Fix TEST_HOST name mismatch.

I saw this error message in the testing section, so I created this PR and submitted my fix. Hope you guys like it.

Before

Copy link
Contributor

@ZachNagengast ZachNagengast left a comment

Choose a reason for hiding this comment

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

Good catch, thank you!

@CongLeSolutionX
Copy link
Contributor Author

Good catch, thank you!

I noticed a failed test in the pipeline. Let me investigate the issue, find a solution, and include it in this PR. Stay tuned!

@ZachNagengast
Copy link
Contributor

The test are currently somewhat indeterminate #100, so this is good to merge now.

@ZachNagengast ZachNagengast merged commit 5e01675 into argmaxinc:main Jul 1, 2024
15 checks passed
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