Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

npalm
Copy link
Member

@npalm npalm commented Apr 30, 2020

To avoid we have to download the github action dist everytime on runner is created we cach the dist in S3.

@npalm npalm requested a review from gertjanmaas April 30, 2020 18:59
import { handle } from './syncer/handler';

module.exports.handler = async (event: any, context: any, callback: any): Promise<any> => {
const statusCode = await handle();
Copy link
Contributor

Choose a reason for hiding this comment

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

StatusCode isn't necessary, this lambda isn't part of the API

@npalm
Copy link
Member Author

npalm commented May 4, 2020

@gertjanmaas solved review comments, also add tagging for resources and update default example. Did not change the used API. The one you suggested is not a public api.

@gertjanmaas gertjanmaas merged commit e219b4a into master May 6, 2020
@gertjanmaas gertjanmaas deleted the feature/sync-github-action-dist branch May 6, 2020 11:26
npalm added a commit that referenced this pull request Aug 5, 2021
# This is the 1st commit message:

Rebase

# This is the commit message #2:

fix: terraform error

# This is the commit message #3:

fix: terraform error

# This is the commit message #4:

fix: test and linter

# This is the commit message #5:

runner labels can be a subset

# This is the commit message #6:

runner labels can be a subset
npalm pushed a commit that referenced this pull request Jan 10, 2025
Co-authored-by: semantic-releaser[bot] <79761287+semantic-releaser[bot]@users.noreply.github.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