Skip to content

Migrate to TypeScript - #3

Open
wescarr wants to merge 1 commit into
mainfrom
feat-typescript-conversion
Open

Migrate to TypeScript#3
wescarr wants to merge 1 commit into
mainfrom
feat-typescript-conversion

Conversation

@wescarr

@wescarr wescarr commented Sep 7, 2022

Copy link
Copy Markdown
Owner
  • Move source to /src from /lib
  • Replace deprecated Buffer.slice method with subarray
  • Add funding entry
  • Better error handling
  • Update README to reflect vercel changes

- Move source to /src from /lib
- Replace deprecated Buffer.slice method with subarray
- Add funding entry
- Better error handling
- Update README to reflect vercel changes
Comment thread tsconfig.json
@@ -0,0 +1,27 @@
{
"extends": "@tsconfig/node16/tsconfig.json",

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You might want to consider @sindresorhus/tsconfig instead.
https://github.com/sindresorhus/tsconfig

Comment thread tsconfig.json
"extends": "@tsconfig/node16/tsconfig.json",
"compilerOptions": {
"lib": [
"es2015",

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Node LTS is 14 which supports es2020

Comment thread package.json
"version": "0.1.0",
"description": "Encrypted environment variables for node done right",
"engines": {
"node": ">=12"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

May as well require Node 14 or later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants