-
Notifications
You must be signed in to change notification settings - Fork 23
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
🎉 Initial Commit: Let's push time to bring this back to life! #15
Conversation
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
@aladdin-add Ready to merge!! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for doing this! I've left a few naming suggestions, and seems the docs(readme) were not updated?
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
@aladdin-add Done with all the changes, Hmm looks like a test failed, NP lemme see |
test/e2e/fixtures/build-default/src/syntax/jsx-import/JSX-import-JSX.jsx
Outdated
Show resolved
Hide resolved
@@ -1,5 +1,7 @@ | |||
import { Template } from './template'; | |||
|
|||
//TODO: REPLACE `@weiran.zsd/tsdx` WITH `@dts-build/dts-build` | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be dts-cli?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah yes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we good to merge?
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
Co-authored-by: 薛定谔的猫 <weiran.zsd@outlook.com>
@aladdin-add Anything else that is left ? or should I merge this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems the PR mainly includes 4 things:
- renaming package tsdx => dts
- removing webiste
- changing parcel to vite
- reformatting some files
what do you think split it to several smaller PRs? (good to keep 1 PR for 1 thing)
export const foo = ( ) => | ||
!! ('bar') | ||
; | ||
export const foo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why changing this file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah this caused an test failure during linting, so to fix the tests I changed it back to faulty file.
Lol Sorry yes I know, I was trying to edit some things using the github web only. Although it committed every one of them. |
closing as fixed. |
First steps so far.