search

amazon-ion / ion-docs

Source for the GitHub Pages for Ion.
https://amazon-ion.github.io/ion-docs/
Apache License 2.0
22 stars 22 forks source link

Update parameter specification for macro signatures #318

Closed popematt closed 4 months ago

popematt commented 4 months ago

Issue #, if available:

Partial fix of #304

Description of changes:

Updates grammar and examples to change macro signatures from [ (name type), ...] to (type::name ...).

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

popematt commented 4 months ago

PR says this is partial fix of #304; can you clarify what's remaining?

Sure. #304 says:

Update signatures to not include types or s-expressions around variables.

I didn't get rid of any types. I did get rid of the s-expressions around each parameter.