Skip to content
This repository has been archived by the owner on Dec 29, 2021. It is now read-only.

Add GitHub actions file #49

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

garg-saurav
Copy link

Issue #48

.github/workflows/workflow.yml Outdated Show resolved Hide resolved
test-server:
runs-on: ubuntu-latest
needs: [download-vectors]
container: docker/compose
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove the container line from each job. For jobs requiring Docker Compose, nothing is required since it's built-in to GitHub Actions. For others such as Node.js, use something like the Node.js GitHub Actions to pull in the latest LTS.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not sure how to use node actions, please review the updated file

- run: sh scripts/deploy.sh
timeout-minutes: 120

pages:
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use the crazy-max/ghaction-github-pages@v2 GitHub Action to host the docs instead of uploading the artifact.

@neelkamath
Copy link
Owner

I'm confused as to what the difference between #49 and #52 is. Is @geekypandey trying to merge his PR into this branch instead?

@geekypandey
Copy link
Contributor

Sorry for the confusion. I made changes to #49, to make it work. @garg-saurav asked for some help for the same.

My bad, I should have mentioned it in the issue.
I hope this clarifies.

@neelkamath
Copy link
Owner

Your changes are for #49 but they're on a separate branch, repo, and PR. You should PR into his branch/repo instead of the master branch on mine.

@geekypandey
Copy link
Contributor

okay!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants