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

Add GitHub actions file#49

Open
garg-saurav wants to merge 3 commits into
neelkamath:masterfrom
garg-saurav:master
Open

Add GitHub actions file#49
garg-saurav wants to merge 3 commits into
neelkamath:masterfrom
garg-saurav:master

Conversation

@garg-saurav

Copy link
Copy Markdown

Issue #48

@neelkamath neelkamath self-requested a review October 2, 2021 11:51
@neelkamath neelkamath added enhancement New feature or request hacktoberfest labels Oct 2, 2021
Comment thread .github/workflows/workflow.yml Outdated
Comment thread .github/workflows/workflow.yml Outdated
test-server:
runs-on: ubuntu-latest
needs: [download-vectors]
container: docker/compose

Copy link
Copy Markdown
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
Copy Markdown
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

@garg-saurav garg-saurav requested a review from neelkamath October 2, 2021 13:55
- run: sh scripts/deploy.sh
timeout-minutes: 120

pages:

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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.

3 participants