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

Gno testing CI #22

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Gno testing CI #22

wants to merge 14 commits into from

Conversation

DIGIX666
Copy link
Contributor

@DIGIX666 DIGIX666 commented Apr 17, 2024

This PR refers to the issue 6
The aim is to have a CI that checks the Test gno files for PR requests

Copy link

netlify bot commented Apr 17, 2024

Deploy Preview for gno-memeland canceled.

Name Link
🔨 Latest commit 70895d4
🔍 Latest deploy log https://app.netlify.com/sites/gno-memeland/deploys/670edc8e0dcbb70008a5d84e

@DIGIX666 DIGIX666 mentioned this pull request Oct 1, 2024
.github/workflow/api.yml Outdated Show resolved Hide resolved
api/r/memeland/memeland_test.gno Outdated Show resolved Hide resolved
@leohhhn
Copy link
Collaborator

leohhhn commented Oct 2, 2024

Also, please add a description to the PR before you make it ready for review :)

@DIGIX666
Copy link
Contributor Author

DIGIX666 commented Oct 3, 2024

The test problem has been solved 👍🏼
But I don't understand my next error when I test the command go run github.com/gnolang/gno/gnovm/cmd/gno test -v ./... locally because when I tried it 6 months ago it worked straight away.

I updated my go.mod but still the same error. Do you think this comes directly from the timezoneinfo.gno file? @leohhhn
image

.github/workflow/api.yml Outdated Show resolved Hide resolved
@leohhhn
Copy link
Collaborator

leohhhn commented Oct 9, 2024

@DIGIX666 if you're running locally, check if you have the latest version of gno/gnovm installed

@DIGIX666
Copy link
Contributor Author

@DIGIX666 if you're running locally, check if you have the latest version of gno/gnovm installed

Yes, I checked but I don't run it locally because I try with this command go run github.com/gnolang/gno/gnovm/cmd/gno test -v ./... and I install the dependencies with the tool.go file for go.mod and go.sum

package tool

import (
	_ "github.com/gnolang/gno/gnovm/cmd/gno"
)

@DIGIX666 DIGIX666 requested a review from leohhhn October 10, 2024 11:15
@leohhhn
Copy link
Collaborator

leohhhn commented Oct 10, 2024

If the CI were to run on this PR, it would show up as an item in the CI box. It's not currently. Can you please read up on the GitHub CI workflow docs and see how to include this properly? 🙏

@DIGIX666
Copy link
Contributor Author

If the CI were to run on this PR, it would show up as an item in the CI box. It's not currently. Can you please read up on the GitHub CI workflow docs and see how to include this properly? 🙏

It's good, now the CI is up and running @leohhhn ✌🏼
But I check the error in the api/p/memeland/memeland_test.gno

@DIGIX666
Copy link
Contributor Author

Hy @leohhhn do you know why the test don't pass here beaucause I try in the main repo and he pass

image

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