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

pkg(go-1.20.1): Add Go 1.20.1 #543

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

e-sp
Copy link
Contributor

@e-sp e-sp commented Feb 27, 2023

user@work ~/s/piston (update-golang)> ./piston run go -l 1.20.1 packages/go/1.20.1/test.*
== Run ==
STDOUT
OK

HexF
HexF previously approved these changes Feb 28, 2023
@HexF HexF enabled auto-merge February 28, 2023 01:24
@e-sp
Copy link
Contributor Author

e-sp commented Feb 28, 2023

Apparently the test failed due to restraints of its environment:

Running packages/go/1.20.1/test.go with runtime=go
== Run ==
STDERR
# math
    runtime: failed to create new OS thread (have 3 already; errno=11)
    runtime: may need to increase max user processes (ulimit -u)
    fatal error: newosproc

Since test.go basically only contains "hello, world", would it make sense to rerun the test? Or do I need to change something?

package main

import "fmt"

func main() {
	fmt.Println("OK")
}

@e-sp
Copy link
Contributor Author

e-sp commented Mar 3, 2023

@HexF anything I can or should do?

auto-merge was automatically disabled March 4, 2023 15:15

Head branch was pushed to by a user without write access

@e-sp e-sp force-pushed the update-golang branch 7 times, most recently from a419238 to 9d69449 Compare March 5, 2023 10:05
@HexF
Copy link
Collaborator

HexF commented Apr 10, 2023

Might need to up the run timeout for tests to pass

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.

None yet

2 participants