这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@ccasin
Copy link

@ccasin ccasin commented May 4, 2023

No description provided.

Signed-off-by: Chris Casinghino <ccasinghino@janestreet.com>
@ccasin ccasin requested a review from goldfirere May 4, 2023 21:31
Copy link

@goldfirere goldfirere left a comment

Choose a reason for hiding this comment

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

Looks pretty good, but it seems easy enough to handle the multiple-attributes case...

Signed-off-by: Chris Casinghino <ccasinghino@janestreet.com>
Copy link

@goldfirere goldfirere left a comment

Choose a reason for hiding this comment

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

Thanks for the explanations!

@ceastlund ceastlund merged commit a5aff07 into oxcaml:jane May 18, 2023
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