-
Notifications
You must be signed in to change notification settings - Fork 471
Fix rewatch compilation of file starting with a lowercase letter #7700
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
Conversation
03cebc3
to
479ac7c
Compare
@tsnobip I don’t believe there’s an issue here. The error seems to be a bit misleading; I was able to successfully build the sample project. Since it’s now configured correctly, the number of files in the shell tests needs to be updated accordingly. |
Okay, I take that back. It was working for me after those file count tweaks on Mac. |
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.
Great work!
3b4a526
to
efdbe42
Compare
let basename = | ||
helpers::file_path_to_compiler_asset_basename(implementation_file_path, &package.namespace); |
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 the fix
This reverts commit 2da1956.
fa765ac
to
0615cc3
Compare
rescript
@rescript/darwin-arm64
@rescript/darwin-x64
@rescript/linux-arm64
@rescript/linux-x64
@rescript/win32-x64
commit: |
#7569 added the test-repo but the test was not being run.