Skip to content

Fix #37#39

Open
sorawee wants to merge 1 commit intoracket:masterfrom
sorawee:fix-37
Open

Fix #37#39
sorawee wants to merge 1 commit intoracket:masterfrom
sorawee:fix-37

Conversation

@sorawee
Copy link
Contributor

@sorawee sorawee commented Jun 9, 2022

Parse 'collection and 'implies when we retrieve the information.
This is done in a way that is as compliant with the existing format
as much as possible for valid values. However, invalid values will be
explicitly marked / quoted so that they cause no trouble.

Two significant changes for valid values:

  • a 'core in 'implies will be further wrapped in a list to distinguish
    a symbol from a string.
  • a 'use-pkg-name will be further wrapped in a list to distinguish
    a symbol from a string.

These two break the compliance, but originally it's not possible to
unambiguously use this information anyway.

Parse 'collection and 'implies when we retrieve the information.
This is done in a way that is as compliant with the existing format
as much as possible for valid values. However, invalid values will be
explicitly marked / quoted so that they cause no trouble.

Two significant changes for valid values:

- a 'core in 'implies will be further wrapped in a list to distinguish
  a symbol from a string.
- a 'use-pkg-name will be further wrapped in a list to distinguish
  a symbol from a string.

These two break the compliance, but originally it's not possible to
unambiguously use this information anyway.
@sorawee
Copy link
Contributor Author

sorawee commented Jun 9, 2022

Please don't merge this yet. I meant this to be a PR for discussion.

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.

1 participant

Comments