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

Setup Ethereum devnet for GHA #69

Merged
merged 8 commits into from
May 27, 2022
Merged

Setup Ethereum devnet for GHA #69

merged 8 commits into from
May 27, 2022

Conversation

jpcenteno
Copy link
Contributor

Issue: #68

@jpcenteno
Copy link
Contributor Author

Note for Monday self:

  • I just modified the main branch permissions for this repo (Because I fucked up by pushing to main When I intended to push into another branch).
  • Now, the CI pipeline, which is what I'm testing here, is not running.
  • Maybe I broke something, maybe it's a problem on the GitHub side. IDK

@jpcenteno jpcenteno marked this pull request as draft May 23, 2022 16:03
@jpcenteno
Copy link
Contributor Author

I got confused because I wasn't getting even a ❌ sign on the left of the commit. It looks that we can see the error log on the actions tab.

The message was:

[Invalid workflow file: .github/workflows/elixir_ci.yml#L27](https://github.com/lambdaclass/ethereum_war_game_tooling/actions/runs/2361011766/workflow)

The workflow is not valid. .github/workflows/elixir_ci.yml (Line: 27, Col: 9): Unexpected value 'command' .github/workflows/elixir_ci.yml (Line: 35, Col: 9): Unexpected value 'hostname'

@jpcenteno jpcenteno changed the title Add Ethereum nodes as GHA CI services Setup Ethereum devnet for GHA May 26, 2022
@jpcenteno jpcenteno marked this pull request as ready for review May 26, 2022 15:50
@jpcenteno jpcenteno merged commit 51c75a3 into main May 27, 2022
@jpcenteno jpcenteno deleted the run-eth-nodes-ci branch May 27, 2022 19:11
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