Skip to content

Commit

Permalink
Remove unused variables
Browse files Browse the repository at this point in the history
  • Loading branch information
davulrich authored and sladg committed Apr 21, 2023
1 parent c4a3b55 commit 8a8fefa
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
2 changes: 0 additions & 2 deletions lib/cdk/utils/cfnCertificate.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,6 @@ export const setupCfnCertificate = (scope: Stack, { domains }: SetupCfnCertifica
// https://github.com/aws/aws-cdk/issues/8934
const multiZoneMap = otherDomains.reduce((acc, curr) => ({ ...acc, [curr.domain]: curr.zone }), {})

const easyCheck = domains.reduce((acc, curr) => ({ ...acc, [curr.domain]: curr.zone.zoneName }), {})

// We need to stick with DNSValidatedCertificate for now, because Certificate construct does not support region specifiation.
// So we would need to manage two different stacks and create certificate in us-east-1 with second stack.
const certificate = new DnsValidatedCertificate(scope, 'Certificate', {
Expand Down
2 changes: 1 addition & 1 deletion lib/utils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ export const executeAsyncCmd = async ({ cmd, path, env }: CommandProps) => {

return new Promise((resolve, reject) => {
console.log('Executing command: ', cmd)
const sh = exec(cmd, { env: { ...process.env, ...env } }, (error, stdout, stderr) => {
const sh = exec(cmd, { env: { ...process.env, ...env } }, (error, stdout) => {
if (error) {
reject(error)
} else {
Expand Down

0 comments on commit 8a8fefa

Please sign in to comment.