-
-
Notifications
You must be signed in to change notification settings - Fork 55
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
feat(attest): add Bun support #1130
base: main
Are you sure you want to change the base?
Conversation
tsconfig.json
Outdated
@@ -3,7 +3,8 @@ | |||
"compilerOptions": { | |||
"allowImportingTsExtensions": true, | |||
"customConditions": ["ark-ts"], | |||
"types": ["mocha", "node"] | |||
"types": ["mocha", "node"], | |||
"allowJs": true |
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.
Can't add allowJS, it screws the repo's typecheck time.
@@ -41,7 +42,7 @@ export const forTypeScriptVersions = ( | |||
try { | |||
if (existsSync(tsPrimaryPath)) unlinkSync(tsPrimaryPath) | |||
|
|||
symlinkSync(targetPath, tsPrimaryPath) | |||
symlinkSync(targetPath, tsPrimaryPath, "junction") |
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.
This is also from the other PR correct? It's possible though that this does break on Windows as the cross-os CI doesn't run across multiple TS versions.
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.
The version that doesn't touch default TS was not on main yet so I did need 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.
Still, I'd rather pnpm testTsVersions
work on Windows as well, so you can leave this in. Just not sure about the build stuff from the other PR.
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.
I'll may drop this commit from this PR later, I did need it to test if it does work
"prettier": "3.3.3", | ||
"@prettier/sync": "0.5.2" | ||
"synckit": "^0.9.1" |
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.
Use "0.9.1"
instead
a10eb2e
to
50c7497
Compare
No description provided.