Skip to content
This repository was archived by the owner on Sep 18, 2023. It is now read-only.

Various small tweaks/updates - #1

Open
jmackie wants to merge 8 commits into
mainfrom
misc-updates
Open

Various small tweaks/updates#1
jmackie wants to merge 8 commits into
mainfrom
misc-updates

Conversation

@jmackie

@jmackie jmackie commented Mar 9, 2022

Copy link
Copy Markdown
  • Collapse the original generated README
  • Remove hello stub
  • MIT -> Unlicense
  • Fixup removing hello
  • Fix type errors
  • Enable strict typechecking
  • prettier
  • Call out Nous specific logic

Comment thread package.json
},
"author": "Nous engineers <engineering@nous.co>",
"license": "MIT"
"license": "Unlicense"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Really wanna do this? It's hardly proprietary

@jmackie jmackie Mar 9, 2022

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

https://unlicense.org/

Unlicense not UNLICENSED (super confusing, seeing as they are polar opposite licenses 😂)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🤦🏼

Comment thread src/functions/index.ts
@@ -1 +0,0 @@
export { default as hello } from "./hello";

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Might as well just delete the file?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants