Skip to content

[All] Add support for [<InlineIfLambda>]#4401

Open
MangelMaxime wants to merge 2 commits intomainfrom
fix-inline-if-lambda
Open

[All] Add support for [<InlineIfLambda>]#4401
MangelMaxime wants to merge 2 commits intomainfrom
fix-inline-if-lambda

Conversation

@MangelMaxime
Copy link
Member

This PR add support for [<InlineIfLambda>]

  1. I needed to store some more information on Fable.AST (once more 🙈)
  2. We need to do the resolution ourself for retrieving the lambda body hopefully I covered the most common cases if not all.

Documentation about this feature is sparse

  1. I add tests for Python, Dart, Rust and JavaScript to make sure I am not breaking the code at runtime
  2. But the real tests for this features are under the integration project, because it allows us to validate the generated source code and not the runtime behaviour

@MangelMaxime MangelMaxime requested review from dbrattli and ncave March 14, 2026 18:23
@github-actions
Copy link
Contributor

Python Type Checking Results (Pyright)

Metric Value
Total errors 18
Files with errors 4
Excluded files 4
New errors ✅ No
Excluded files with errors (4 files)

These files have known type errors and are excluded from CI. Remove from pyrightconfig.ci.json as errors are fixed.

File Errors Status
temp/tests/Python/test_applicative.py 12 Excluded
temp/tests/Python/test_hash_set.py 3 Excluded
temp/tests/Python/test_nested_and_recursive_pattern.py 2 Excluded
temp/tests/Python/fable_modules/thoth_json_python/encode.py 1 Excluded

@MangelMaxime
Copy link
Member Author

I believe the CI for JavaScript broke because the changes made to Fable.AST are not backward compatible.

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

Successfully merging this pull request may close these issues.

2 participants