Skip to content

Comments

Remove function surjection#1887

Merged
lewisjkl merged 2 commits intoseries/0.19from
remove-function-surjection
Feb 18, 2026
Merged

Remove function surjection#1887
lewisjkl merged 2 commits intoseries/0.19from
remove-function-surjection

Conversation

@lewisjkl
Copy link
Contributor

@lewisjkl lewisjkl commented Feb 13, 2026

This avoids issues where it is used as an implicit conversion in Scala 2. Closes #1884.

PR Checklist (not all items are relevant to all PRs)

  • Added unit-tests (for runtime code)
  • Added bootstrapped code + smoke tests (when the rendering logic is modified)
  • Added build-plugins integration tests (when reflection loading is required at codegen-time)
  • Added alloy compliance tests (when simpleRestJson protocol behaviour is expanded/updated)
  • Updated dynamic module to match generated-code behaviour
  • Added documentation
  • Updated changelog

@lewisjkl lewisjkl changed the base branch from series/0.18 to series/0.19 February 13, 2026 22:54
@lewisjkl lewisjkl merged commit ec25ae8 into series/0.19 Feb 18, 2026
10 checks passed
@lewisjkl lewisjkl deleted the remove-function-surjection branch February 18, 2026 17:50
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.

0.19: Surjection shouldn't extend Function

2 participants