Skip to content

Add json property to metadata result#303

Closed
jaraco wants to merge 4 commits intomainfrom
json-subclass
Closed

Add json property to metadata result#303
jaraco wants to merge 4 commits intomainfrom
json-subclass

Conversation

@jaraco
Copy link
Copy Markdown
Member

@jaraco jaraco commented Apr 2, 2021

Alternate to #301, this approach wraps the metadata result in an object with a json property. This approach has the advantage that it doesn't add anything to the namespace and gives consumers access to the json on their existing object.

It has the downside of adding new demands on the PackageMetadata protocol (namely that it must have a .json property).

@jaraco jaraco force-pushed the main branch 2 times, most recently from 591b2c3 to 5ca9bc7 Compare April 6, 2021 12:57
@jaraco jaraco closed this Apr 18, 2021
@jaraco jaraco mentioned this pull request Apr 18, 2021
@jaraco jaraco deleted the json-subclass branch April 18, 2021 14:56
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