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

cracked project includes unrelated cs files from node_modules #3977

Open
joprice opened this issue Dec 16, 2024 · 2 comments
Open

cracked project includes unrelated cs files from node_modules #3977

joprice opened this issue Dec 16, 2024 · 2 comments

Comments

@joprice
Copy link
Contributor

joprice commented Dec 16, 2024

Description

When taking a look at the project metadata in fable_modules/project_cracked.json, I found that some files like node_modules/node-gyp/lib/Find-VisualStudio.cs are mentioned in SourcePaths, without being included anywhere in the project. It seems there may be a glob for the fable_modules library that is unintentionally including fsharp files from node_modules. Not sure if this causes issues, but it seems incorrect. Perhaps there's a flag to suppress this? Not sure if it's intentional to pull in fsharp sources from npm dependencies.

Related information

  • Fable version: 4.24.0
  • Operating system: OSX
@MangelMaxime
Copy link
Member

Does it also happens when passing --test:MSBuildCracker to Fable CLI ?

@joprice
Copy link
Contributor Author

joprice commented Dec 19, 2024

I flipped the flag and ran find . -name project_cracked_debug.json | xargs jq '.' | grep node_modules and it is definitely no longer happening with the MSBuildCracker 👍. If that's going to be the default in Fable 5, then no need to investigate.

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

No branches or pull requests

2 participants