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: typo in the HTTP port, for #31 #35

Merged
merged 1 commit into from
Jun 5, 2024
Merged

Conversation

stasadev
Copy link
Member

@stasadev stasadev commented Jun 5, 2024

The Issue

I noticed extra : which breaks the HTTP URL.

How This PR Solves The Issue

Removes it.

Manual Testing Instructions

Automated Testing Overview

Related Issue Link(s)

Release/Deployment Notes

@tyler36
Copy link
Collaborator

tyler36 commented Jun 5, 2024

Think the tests failed due to move to official status and requiring #34 to be merge.

I just did that so the tests should pass now.

@stasadev stasadev force-pushed the 20240605_stasadev_http_port branch from 875df87 to b8429d4 Compare June 5, 2024 09:41
@stasadev
Copy link
Member Author

stasadev commented Jun 5, 2024

I did this in the repo settings to enable the button for rebase:

image

And rebased the branch, it should be good now.

@tyler36 tyler36 merged commit 53d9bed into main Jun 5, 2024
2 checks passed
@tyler36 tyler36 deleted the 20240605_stasadev_http_port branch June 5, 2024 23:48
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