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: runner url in contributing.md #254

Merged
merged 1 commit into from
Sep 17, 2022
Merged

fix: runner url in contributing.md #254

merged 1 commit into from
Sep 17, 2022

Conversation

pfiaux
Copy link
Contributor

@pfiaux pfiaux commented Sep 16, 2022

I had this strange error trying to run the runner for a PR:

15:56:33.469	error	failed to connect	{"error": "dial tcp: lookup github.bass-loop.org on ....1:53: no such host"}
dial: 1 error occurred:
	* dial tcp: lookup github.bass-loop.org on ...1:53: no such host

Looks like the url wasn't correct in the example.

I had this strange error trying to run the runner for a PR:
```
15:56:33.469	error	failed to connect	{"error": "dial tcp: lookup github.bass-loop.org on ....1:53: no such host"}
dial: 1 error occurred:
	* dial tcp: lookup github.bass-loop.org on ...1:53: no such host
```
Looks like the url wasn't correct in the example.
@vito
Copy link
Owner

vito commented Sep 17, 2022

Whoops, nice catch. Thanks!

Not sure why all the builds failed for this. Was the bass --runner command terminated in the middle of it? Either way this is obviously OK to merge so I'll go ahead and do that.

@vito vito merged commit ba4e86b into vito:main Sep 17, 2022
@pfiaux
Copy link
Contributor Author

pfiaux commented Sep 18, 2022

I had the runner in the foreground, when I realized it wasn't going to pickup the commits from #242 I killed it without realizing it was running these. If I remember correctly the output only shows the health checks/keep alive pings from the server, but I didn't see any "currently running ..." so I didn't realize it was running jobs 😬.

@pfiaux pfiaux deleted the patch-1 branch September 18, 2022 09:00
@vito
Copy link
Owner

vito commented Sep 18, 2022

Yeah true, I've been thinking of having it show all the output for everything it's running. It's definitely a bit too quiet.

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