search

openziti / ziti-webhook-action

Github Action to post a Webhook over a Ziti network
Apache License 2.0
3 stars 0 forks source link

add input 'data' (dict) to be merged with GitHub context payload #10

Closed qrkourier closed 2 years ago

qrkourier commented 2 years ago

@smilindave26 Oh, OK. I think you're saying that I need to reflect my changes from dist/index.js in top-level index.js, and then run one or more npm commands to locally build (combine the SDK with index.js?) which checked-in artifact will be the dist sub-directory. I believe I'll need to answer this question to proceed:

What is/are the npm command(s) that you would typically use to test/build/install the top-level source files e.g. index.js? npm --help offers me subcommands like test and install, but I get a lot of errors for npm install. Does this step only work on MacOS in light of the fact that only the Darwin build of the SDK is checked in under dist?

I'll add a BUILD.md or CONTRIBUTING.md or both based on what I learn here.

smilindave26 commented 2 years ago

Take a look at the CI.yml workflow, especially ncc, for building dist/