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

fix: fixes the issue with the github action checkout #7

Merged
merged 1 commit into from
Dec 8, 2022

Conversation

r3dsm0k3
Copy link
Collaborator

@r3dsm0k3 r3dsm0k3 commented Dec 5, 2022

By submitting this pull request, I confirm that my contribution is made under the terms of MIT license.

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

For checking out the public repository of goloop, we are using the pre-defined action component called actions/checkout. However, they seems to have a bug where the GitHub token is being required even while cloning a public repo. This happens even when running locally with act (issue)

The suggested fix just does a normal git clone operation which seems to work just fine.

Fixes:

#5

@r3dsm0k3 r3dsm0k3 requested a review from ant4g0nist December 5, 2022 17:25
Copy link
Collaborator

@ant4g0nist ant4g0nist left a comment

Choose a reason for hiding this comment

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

LGTM

@r3dsm0k3 r3dsm0k3 merged commit 1cc76d4 into icon-community:dev-0.1.0 Dec 8, 2022
@r3dsm0k3 r3dsm0k3 deleted the fix/checkout-action branch December 8, 2022 18:55
r3dsm0k3 added a commit that referenced this pull request Jan 15, 2023
* feat: adds init, install and compile commands
changes:
- adds install command to pull in Docker images
- adds init command that initializes a Drogon project
- adds compile command that compiles the initialised Drogon project
- adds pulling in biolerplates/scaffolding
- adds workflows to build Drogon and Gochain images

* fix: fixes github action files

* fix: fixes Dockerfile and github actions

* fix: adds icon/gradle*

* fix: fixes github actions

* fix: updates github actions to create packages

* fix: updates github actions to create packages

* fix: fixes github actions to fix permissions

* adds docker hub push

* fix: fixes the github workflow

* fix: fixes the push to the community page

* feat: adds support for testing, optimizing
changes:
- adds 4 new commands
- test command to run tests
- optimize command to run optimizeJar tasks
- deploy command to run deployJar tasks
- gradlew subcommand that exposes gradle wrapper to drogon
- compile and test command now return CI/CD friendly exit codes
- install now pulls docker images from docker hub instead of ghcr.io

* fix: adds Docker debian image
Changes:
- adds Debian Docker image and improves goloop support

* fix: fix github actions

* feat: adds sandbox, goloop subcommands
changes:
- adds sandbox start and sandbox stop subcommands
- adds goloop command
- adds keystore command to generate keystores
- restructures code

* fix: fixes github actions

* fix: fixes github actions

* fix: fixes #5

* fix: fixes gochain image

* chore: adding dev dependencies and some run scripts to make it easier

* chore: removes yarn lock; lets use npm

* chore: linter fixes

* chore: removes the @typescript-eslint/no-explicit-any; FIX later

* chore: fixes the linter warnings; disabled the rules for irrelevant ones

* feat: adds ability to generate binary with pkg

* fix: fixes the issue with the github action checkout (#7)

* fix: gochain build err on docker file (#9)

* fix: fixes the issue where the gochain was not getting built inside dockerfile

* fix: fixes the image tagging

Co-authored-by: ant4g0nist <ant4g0nist@users.noreply.github.com>
Co-authored-by: Chaithu <3500559+ant4g0nist@users.noreply.github.com>
r3dsm0k3 added a commit that referenced this pull request Jan 16, 2023
* prepare tsconfig for publish

* adds run scripts

* linter error fix

* 0.1.1

* Revert "0.1.1"

This reverts commit f83ceba.

* 0.1.1

* 0.1.2

* ability to read the lib version from the package

* Milestone 2 (#13)

* feat: adds init, install and compile commands
changes:
- adds install command to pull in Docker images
- adds init command that initializes a Drogon project
- adds compile command that compiles the initialised Drogon project
- adds pulling in biolerplates/scaffolding
- adds workflows to build Drogon and Gochain images

* fix: fixes github action files

* fix: fixes Dockerfile and github actions

* fix: adds icon/gradle*

* fix: fixes github actions

* fix: updates github actions to create packages

* fix: updates github actions to create packages

* fix: fixes github actions to fix permissions

* adds docker hub push

* fix: fixes the github workflow

* fix: fixes the push to the community page

* feat: adds support for testing, optimizing
changes:
- adds 4 new commands
- test command to run tests
- optimize command to run optimizeJar tasks
- deploy command to run deployJar tasks
- gradlew subcommand that exposes gradle wrapper to drogon
- compile and test command now return CI/CD friendly exit codes
- install now pulls docker images from docker hub instead of ghcr.io

* fix: adds Docker debian image
Changes:
- adds Debian Docker image and improves goloop support

* fix: fix github actions

* feat: adds sandbox, goloop subcommands
changes:
- adds sandbox start and sandbox stop subcommands
- adds goloop command
- adds keystore command to generate keystores
- restructures code

* fix: fixes github actions

* fix: fixes github actions

* fix: fixes #5

* fix: fixes gochain image

* chore: adding dev dependencies and some run scripts to make it easier

* chore: removes yarn lock; lets use npm

* chore: linter fixes

* chore: removes the @typescript-eslint/no-explicit-any; FIX later

* chore: fixes the linter warnings; disabled the rules for irrelevant ones

* feat: adds ability to generate binary with pkg

* fix: fixes the issue with the github action checkout (#7)

* fix: gochain build err on docker file (#9)

* fix: fixes the issue where the gochain was not getting built inside dockerfile

* fix: fixes the image tagging

Co-authored-by: ant4g0nist <ant4g0nist@users.noreply.github.com>
Co-authored-by: Chaithu <3500559+ant4g0nist@users.noreply.github.com>

* adds installation instructions

* adds package name

* adds github workflows

* updates version from package

* chore: bumps version

Co-authored-by: coconut-bunch <ajith.c@outlook.com>
Co-authored-by: ant4g0nist <ant4g0nist@users.noreply.github.com>
Co-authored-by: Chaithu <3500559+ant4g0nist@users.noreply.github.com>
r3dsm0k3 added a commit that referenced this pull request Jan 16, 2023
* feat: adds init, install and compile commands
changes:
- adds install command to pull in Docker images
- adds init command that initializes a Drogon project
- adds compile command that compiles the initialised Drogon project
- adds pulling in biolerplates/scaffolding
- adds workflows to build Drogon and Gochain images

* fix: fixes github action files

* fix: fixes Dockerfile and github actions

* fix: adds icon/gradle*

* fix: fixes github actions

* fix: updates github actions to create packages

* fix: updates github actions to create packages

* fix: fixes github actions to fix permissions

* adds docker hub push

* fix: fixes the github workflow

* fix: fixes the push to the community page

* feat: adds support for testing, optimizing
changes:
- adds 4 new commands
- test command to run tests
- optimize command to run optimizeJar tasks
- deploy command to run deployJar tasks
- gradlew subcommand that exposes gradle wrapper to drogon
- compile and test command now return CI/CD friendly exit codes
- install now pulls docker images from docker hub instead of ghcr.io

* fix: adds Docker debian image
Changes:
- adds Debian Docker image and improves goloop support

* fix: fix github actions

* feat: adds sandbox, goloop subcommands
changes:
- adds sandbox start and sandbox stop subcommands
- adds goloop command
- adds keystore command to generate keystores
- restructures code

* fix: fixes github actions

* fix: fixes github actions

* fix: fixes #5

* fix: fixes gochain image

* chore: adding dev dependencies and some run scripts to make it easier

* chore: removes yarn lock; lets use npm

* chore: linter fixes

* chore: removes the @typescript-eslint/no-explicit-any; FIX later

* chore: fixes the linter warnings; disabled the rules for irrelevant ones

* feat: adds ability to generate binary with pkg

* fix: fixes the issue with the github action checkout (#7)

* fix: gochain build err on docker file (#9)

* fix: fixes the issue where the gochain was not getting built inside dockerfile

* fix: fixes the image tagging

* feat: npm package release (#14)

* prepare tsconfig for publish

* adds run scripts

* linter error fix

* 0.1.1

* Revert "0.1.1"

This reverts commit f83ceba.

* 0.1.1

* 0.1.2

* ability to read the lib version from the package

* Milestone 2 (#13)

* feat: adds init, install and compile commands
changes:
- adds install command to pull in Docker images
- adds init command that initializes a Drogon project
- adds compile command that compiles the initialised Drogon project
- adds pulling in biolerplates/scaffolding
- adds workflows to build Drogon and Gochain images

* fix: fixes github action files

* fix: fixes Dockerfile and github actions

* fix: adds icon/gradle*

* fix: fixes github actions

* fix: updates github actions to create packages

* fix: updates github actions to create packages

* fix: fixes github actions to fix permissions

* adds docker hub push

* fix: fixes the github workflow

* fix: fixes the push to the community page

* feat: adds support for testing, optimizing
changes:
- adds 4 new commands
- test command to run tests
- optimize command to run optimizeJar tasks
- deploy command to run deployJar tasks
- gradlew subcommand that exposes gradle wrapper to drogon
- compile and test command now return CI/CD friendly exit codes
- install now pulls docker images from docker hub instead of ghcr.io

* fix: adds Docker debian image
Changes:
- adds Debian Docker image and improves goloop support

* fix: fix github actions

* feat: adds sandbox, goloop subcommands
changes:
- adds sandbox start and sandbox stop subcommands
- adds goloop command
- adds keystore command to generate keystores
- restructures code

* fix: fixes github actions

* fix: fixes github actions

* fix: fixes #5

* fix: fixes gochain image

* chore: adding dev dependencies and some run scripts to make it easier

* chore: removes yarn lock; lets use npm

* chore: linter fixes

* chore: removes the @typescript-eslint/no-explicit-any; FIX later

* chore: fixes the linter warnings; disabled the rules for irrelevant ones

* feat: adds ability to generate binary with pkg

* fix: fixes the issue with the github action checkout (#7)

* fix: gochain build err on docker file (#9)

* fix: fixes the issue where the gochain was not getting built inside dockerfile

* fix: fixes the image tagging

Co-authored-by: ant4g0nist <ant4g0nist@users.noreply.github.com>
Co-authored-by: Chaithu <3500559+ant4g0nist@users.noreply.github.com>

* adds installation instructions

* adds package name

* adds github workflows

* updates version from package

* chore: bumps version

Co-authored-by: coconut-bunch <ajith.c@outlook.com>
Co-authored-by: ant4g0nist <ant4g0nist@users.noreply.github.com>
Co-authored-by: Chaithu <3500559+ant4g0nist@users.noreply.github.com>

Co-authored-by: ant4g0nist <ant4g0nist@users.noreply.github.com>
Co-authored-by: Chaithu <3500559+ant4g0nist@users.noreply.github.com>
Co-authored-by: coconut-bunch <ajith.c@outlook.com>
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