-
Notifications
You must be signed in to change notification settings - Fork 234
migrate to typescript & node.js-compatible es modules #246
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
The head ref may contain hidden characters: "\u00F8"
Merged
Changes from 48 commits
Commits
Show all changes
52 commits
Select commit
Hold shift + click to select a range
697df6f
esm: port code to `--experimental-modules`
zackschuster cc08daa
rollup: remove require statement
zackschuster 50b9ec1
typescript: naive zero-build-error port
zackschuster 3bb741b
smtp: complete typescript conversion
zackschuster d57ecc2
test: convert to typescript w/ ava
zackschuster ae38188
chore: run tests with `--experimental-modules`
zackschuster 8993c49
chore: naively upgrade all deps
zackschuster 16a23e9
chore: update linting
zackschuster aa7239e
chore: update typescript config
zackschuster 6f1a0e7
chore: lint root files
zackschuster f3192af
rollup: convert to typescript & output cjs & esm bundles
zackschuster 7062071
chore: set up dual package exports
zackschuster 1727412
smtp: repair differences with pre-conversion code
zackschuster 5d34773
test: don't destructure cjs modules
zackschuster 23fefeb
test: use `.cb` for everything
zackschuster 0c045e1
chore: update gitignore
zackschuster c1eb2ee
build: include declaration files in output
zackschuster b97fda8
chore: fix nits
zackschuster ff606d2
chore: upgrade deps
zackschuster 288ec6b
test: get (mostly) working
zackschuster e3cd104
test: set NODE_TLS_REJECT_UNAUTHORIZED from config
zackschuster a1d8b81
chore: disable module mode
zackschuster dba555c
build: update distribution config
zackschuster 03d4f0d
chore: remove old bundle
zackschuster 9376d37
chore: gitignore new bundle outputs
zackschuster b6723b4
chore: move prettier config to separate file
zackschuster 676c541
chore: move ava config to separate file
zackschuster 50c3789
test: remove timeouts
zackschuster 1322ba2
build: don't remove comments
zackschuster 93f23cd
chore: move type imports from inline statements
zackschuster 106e5ba
smtp: strongly type addressparser & emailjs-mime-codec
zackschuster 42a632a
smtp: add Connection to bare SMTP object names
zackschuster e0e0f83
chore: update readme
zackschuster f7bb694
test: un-skip failing unit
zackschuster 962e3b0
duty: retire travis
zackschuster efe9daa
api: lowercase smtp namespace export
zackschuster a537e35
chore: fix nits in manifest
zackschuster dc1bd28
smtp: strongly type attachment checks
zackschuster 3c1681c
smtp: stricten property access modifiers
zackschuster ab7ecaf
smtp: specify method access modifiers
zackschuster f9d8388
smtp: remove reference to unpublished types
zackschuster a945688
chore: remove unused dep
zackschuster e6f4cc2
test: use t.is instead of t.deepEqual
zackschuster 9d80365
lint: enable @typescript-eslint plugin
zackschuster 4d0b531
chore: upgrade deps
zackschuster bb34667
test: fix mailparser timing out smtp
zackschuster 92e4d7f
build: ready bundle configuration
zackschuster ce4683d
test: enable transparent cjs bundle testing
zackschuster 4bc4d02
chore: add type hint to manifest
zackschuster 408df22
chore: make export paths relative
zackschuster 652b021
chore: upgrade deps
zackschuster 812353a
chore: convert prettier config to json
zackschuster File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,7 +1,7 @@ | ||
| /.vscode/ | ||
| /node_modules/ | ||
| /test/config.js | ||
| /npm-debug.log | ||
| .vscode | ||
| node_modules | ||
|
|
||
| *.log | ||
| *.swp | ||
| *.swo | ||
| *~ | ||
|
|
||
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| export default { | ||
| files: ['test/*.ts'], | ||
| extensions: ['ts'], | ||
| require: ['./email.test.ts'], | ||
| environmentVariables: { | ||
| NODE_TLS_REJECT_UNAUTHORIZED: '0', | ||
| }, | ||
| }; |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.