Skip to content

Fix anonymous function definitions#41

Open
reiniscirpons wants to merge 1 commit intogap-packages:masterfrom
reiniscirpons:patch-1
Open

Fix anonymous function definitions#41
reiniscirpons wants to merge 1 commit intogap-packages:masterfrom
reiniscirpons:patch-1

Conversation

@reiniscirpons
Copy link

This PR fixes an issue with the get and test functions. When defining an anonymous nullary function, the proper syntax is {} -> foo; to indicate that no arguments are passed.

This PR fixes an issue with the `get` and `test` functions. When defining an anonymous nullary function, the proper syntax is `{} -> foo;` to indicate that no arguments are passed.
@fingolfin
Copy link
Member

fingolfin commented Jan 26, 2026

If you are interested in working on or with this package, let me know and I can also give you write access. Right now it is unmaintained :/

@codecov
Copy link

codecov bot commented Jan 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 42.98%. Comparing base (e6dcbc2) to head (3385f19).

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #41   +/-   ##
=======================================
  Coverage   42.98%   42.98%           
=======================================
  Files           2        2           
  Lines         449      449           
  Branches       69       69           
=======================================
  Hits          193      193           
  Misses        234      234           
  Partials       22       22           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@reiniscirpons
Copy link
Author

Hi Max, sadly not a user of ZeroMQInterface myself. This issue just came up when I was working on the tree-sitter grammar yesterday and it was flagged up by the automated corpus tests for the grammar (i.e. these files were not parsing), so I felt I might as well fix it.

Copy link
Member

@olexandr-konovalov olexandr-konovalov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks - I'm also approving. @fingolfin please feel free to make a release. I have some interest in this package since it's a dependency of Jupyter Kernel, but at the moment I am just happy that it works.

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.

3 participants