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

Update custom elements manifest #50

Merged
merged 10 commits into from
Mar 16, 2024
Merged

Conversation

clhenrick
Copy link
Owner

@clhenrick clhenrick commented Mar 16, 2024

Makes some updates to files to more correctly document the custom element using the custom elements manifest

recreates / updates the custom-elements.json with the following changes:

  • docs for CSS variables
  • docs for two slots
  • docs for component summary
  • ignores documenting private class members on theColorLegendElement class
  • ignores documenting non-essential modules (everything but color-legend-element.ts and constants.ts)

Other:

  • updates the package.json's files field to include custom-elements.json in the package root when publishing
  • updates the package.json's files field to ignore the build/tests/ directory so that it's excluded when publishing

@clhenrick clhenrick changed the title Update custom elements json Update custom elements manifest Mar 16, 2024
- include custom-elements.json
- exclude build/test/ dir
Copy link
Owner Author

@clhenrick clhenrick left a comment

Choose a reason for hiding this comment

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

LGTM

@clhenrick clhenrick merged commit 46c1a92 into main Mar 16, 2024
1 check passed
@clhenrick clhenrick deleted the update-custom-elements-json branch March 16, 2024 17:20
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