Skip to content
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

WIP Convert to ts #8

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Conversation

devinrhode2
Copy link
Contributor

Lot of work/cleanup to do here still. Just wanted to share WIP as I may not get back to this for a few weekends.

I was having a time with the types, so started inling all the functions, so see what the natural types are, convert/type case values to something better, than pass pure data into utility functions

Most recent commit is going to be the most interesting

Probably this should live along side some JS version, since likely whatever I come up with will be extra "typescript-y"

devinrhode2 added 12 commits August 8, 2021 14:23
Given a standard .gitattributes file (just added in previous commit)
after doing a fresh clone of this repo on mac with vanilla git v2.32.0,
git status shows:

	modified:   package-lock.json
	modified:   package.json

$ git diff
warning: CRLF will be replaced by LF in package-lock.json.
warning: CRLF will be replaced by LF in package.json.

diff --git a/package-lock.json b/package-lock.json
Binary files a/package-lock.json and b/package-lock.json differ

diff --git a/package.json b/package.json
-{
-  "name": "solid-todomvc",
...

Seems like best practice is to add this to your global ~/.gitconfig:

  [core]
    attributesfile = ~/.gitattributes_global
Co-authored-by: ts-migrate <>
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.

1 participant