Skip to content

Commit

Permalink
remove isEmailAddress because it breaks tests (and probably irrelevant?)
Browse files Browse the repository at this point in the history
  • Loading branch information
Schniz committed Sep 30, 2024
1 parent 82a2ff8 commit 4ac6f73
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 6 deletions.
5 changes: 0 additions & 5 deletions src/sdk.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ import { createScheduledHandler } from './instrumentation/scheduled.js'
//@ts-ignore
import * as versions from '../versions.json'
import { createEmailHandler } from './instrumentation/email.js'
import { EmailMessage } from 'cloudflare:email'

type FetchHandler = ExportedHandlerFetchHandler<unknown, unknown>
type ScheduledHandler = ExportedHandlerScheduledHandler<unknown>
Expand All @@ -35,10 +34,6 @@ export function isAlarm(trigger: Trigger): trigger is 'do-alarm' {
return trigger === 'do-alarm'
}

export function isEmailMessage(trigger: Trigger): trigger is ForwardableEmailMessage {
return !!(trigger instanceof EmailMessage && trigger.headers && trigger.forward)
}

const createResource = (config: ResolvedTraceConfig): Resource => {
const workerResourceAttrs = {
'cloud.provider': 'cloudflare',
Expand Down
1 change: 0 additions & 1 deletion tsup.config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,5 +6,4 @@ export default defineConfig({
dts: true,
clean: true,
sourcemap: true,
external: ['cloudflare:email'],
})

0 comments on commit 4ac6f73

Please sign in to comment.