Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: add schema example #91

Merged

Conversation

thepassle
Copy link
Collaborator

No description provided.

README.md Outdated
customElements.define('my-element', MyElement);
```

The output manifest would look like:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe "A manifest could look like"? Or also mention this example is in a package called my-project or that it has a readme?

There are things like readme being blank that a tool might fill in for an element, it'd be nice to be clear that if its tool generated the tool might have lots of opinions and features for how to fill out the manifest.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

"Could" sounds ambiguous to me, we're specifying the schema in this repo so it would have to look exactly like the example shown

Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't think that necessarily true. For instance, this could contain source references for all the declarations. disabled sounds like a boolean, so the field could get a type. Conceivably someone could add extra descriptions to the manifest that aren't in the JS. Tools will have different features for extracting types, descriptions, etc...

Copy link
Collaborator Author

@thepassle thepassle Dec 7, 2021

Choose a reason for hiding this comment

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

For instance, this could contain source references for all the declarations

I wish I knew what a source reference is 😄

disabled sounds like a boolean, so the field could get a type.

Not all code is typed

Copy link
Collaborator

Choose a reason for hiding this comment

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

I thought I saw an issue from you about SourceReference documentation, but I don't see it now. Do you know where it is?

All values have a type and a developer could certainly write a type in a hand-written manifest. Point is that "could" really is the accurate descriptor here. There are multiple valid manifests per element.

README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
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.

None yet

2 participants