Skip to content

Commit

Permalink
build(serverless): Remove unused build check (#10829)
Browse files Browse the repository at this point in the history
It seems this is failing (sometimes?) and I don't think we need it
anymore.

e.g. see
https://github.com/getsentry/sentry-javascript/actions/runs/8067785899/job/22039493880?pr=10825
  • Loading branch information
mydea authored Feb 27, 2024
1 parent e8c523d commit 2a8ef40
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 164 deletions.
5 changes: 0 additions & 5 deletions packages/serverless/scripts/buildLambdaLayer.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ import * as childProcess from 'child_process';
import * as fs from 'fs';
import * as rimraf from 'rimraf';

import { ensureBundleBuildPrereqs } from '../../../scripts/ensure-bundle-deps';
import { version } from '../package.json';

/**
Expand All @@ -16,10 +15,6 @@ function run(cmd: string, options?: childProcess.ExecSyncOptions): string {

async function buildLambdaLayer(): Promise<void> {
// Create the main SDK bundle
// TODO: Check if we can get rid of this, after the lerna 6/nx update??
await ensureBundleBuildPrereqs({
dependencies: ['@sentry/utils', '@sentry/core', '@sentry/node-experimental'],
});
run('yarn rollup --config rollup.aws.config.mjs');

// We build a minified bundle, but it's standing in for the regular `index.js` file listed in `package.json`'s `main`
Expand Down
159 changes: 0 additions & 159 deletions scripts/ensure-bundle-deps.ts

This file was deleted.

0 comments on commit 2a8ef40

Please sign in to comment.