feat: add TypeScript configuration detection to file systems and update attw behavior #59

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.

Sign up for GitHub

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

Jump to bottom

Merged

43081j merged 7 commits into e18e:main from dreyfus92:feat/attw-only-ts

Aug 19, 2025

Contributor

dreyfus92 commented Aug 7, 2025 •

edited

Loading

closes: #40


          feat: add TypeScript configuration detection to file systems and upda…

18facd8

…te attw behavior

pkg-pr-new bot commented Aug 7, 2025 •

edited

Loading

Open in StackBlitz

npm i https://pkg.pr.new/e18e/cli/@e18e/cli@59

commit: e71d0a8

43081j reviewed

View reviewed changes

src/local-file-system.ts Outdated Show resolved Hide resolved

43081j reviewed

View reviewed changes

src/test/attw-typescript-config.test.ts Outdated Show resolved Hide resolved

43081j reviewed

View reviewed changes

src/test/typescript-config.test.ts Outdated Show resolved Hide resolved


          Apply suggestions from @43081j's review.

d6472a4

43081j reviewed

View reviewed changes

src/tarball-file-system.ts Outdated Show resolved Hide resolved

dreyfus92 added 2 commits

August 8, 2025 13:55


          refactor: remove hasTypeScriptConfig method and replace with fileExis…

bc9b751

…ts check for tsconfig.json


          format

31cf4ac

Contributor Author

dreyfus92 commented Aug 11, 2025

@43081j is this good to go? 👀

43081j reviewed

View reviewed changes

src/test/analyze/attw.test.ts Outdated Show resolved Hide resolved

43081j reviewed

View reviewed changes

src/test/analyze/attw.test.ts Outdated Show resolved Hide resolved

43081j reviewed

View reviewed changes

src/test/local-file-system.test.ts Outdated Show resolved Hide resolved

43081j reviewed

View reviewed changes

src/test/local-file-system.test.ts Outdated Show resolved Hide resolved


          Apply suggestions from @43081j's review.

Contributor Author

dreyfus92 commented Aug 19, 2025

Thank you for the review @43081j 🤘🏻

43081j reviewed

View reviewed changes

src/test/local-file-system.test.ts Outdated Show resolved Hide resolved

43081j reviewed

View reviewed changes

src/test/local-file-system.test.ts Show resolved Hide resolved

43081j reviewed

View reviewed changes

src/test/local-file-system.test.ts Outdated Show resolved Hide resolved

43081j reviewed

View reviewed changes

src/test/local-file-system.test.ts Outdated Show resolved Hide resolved

43081j reviewed

View reviewed changes

src/test/tarball-file-system.test.ts Outdated Show resolved Hide resolved


          Apply more suggestions.

55e2bbb

43081j reviewed

View reviewed changes

src/test/tarball-file-system.test.ts Outdated Show resolved Hide resolved


          update: update test descriptions for file existence checks in Tarball…

e71d0a8

…FileSystem

43081j approved these changes

View reviewed changes

43081j merged commit e159e4b into e18e:main

5 checks passed

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

Labels

None yet