Skip to content

Commit

Permalink
Merge pull request #47 from ably/actually-make-inputs-required
Browse files Browse the repository at this point in the history
Enforce that required inputs are supplied
  • Loading branch information
QuintinWillison authored Sep 26, 2022
2 parents 5ca4d72 + 9cce54d commit 30dd8a1
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ if (typeof githubRef !== 'string') {
process.exit(1);
}

const githubToken = core.getInput('githubToken');
const githubToken = core.getInput('githubToken', {required: true});
const octokit = getOctokit(githubToken);

const githubEvent = JSON.parse(fs.readFileSync(githubEventPath, 'utf8'));
Expand Down Expand Up @@ -54,7 +54,7 @@ const createRef = (githubRef: string) => {
const ref = createRef(githubRef)

const s3BucketName = 'sdk.ably.com';
const sourcePath = path.resolve(core.getInput('sourcePath'));
const sourcePath = path.resolve(core.getInput('sourcePath', {required: true}));

// Optional artifactName:
// - The getInput() method calls trim() for us by default (trimWhitespace: true)
Expand Down

0 comments on commit 30dd8a1

Please sign in to comment.