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

chore(astro): Add Astro integration keywords #9193

Merged
merged 2 commits into from
Oct 13, 2023

Conversation

Lms24
Copy link
Member

@Lms24 Lms24 commented Oct 6, 2023

Adding specific keywords to an NPM package makes them show up in Astro's integration library page. This PR adds them to the Astro package.json.

When we're ready we should also file an issue with Astro to add a Sentry logo in the library.

Separate PR because I'd only merge this once we're comfortable with publishing.

ref #9182

@Lms24 Lms24 force-pushed the lms/astro-integration-keywords branch from 2198f3f to e49337c Compare October 6, 2023 15:05
Base automatically changed from lms/astro-boilerplate-package to develop October 6, 2023 15:34
@AbhiPrasad
Copy link
Member

Let's ask astro if we can add a monitoring/developer tooling category for their integrations cc @HazAT

@Lms24 Lms24 force-pushed the lms/astro-integration-keywords branch from e49337c to 43c994c Compare October 9, 2023 11:00
"keywords": [
"withastro",
"astro-component",
"analytics",
Copy link
Member Author

Choose a reason for hiding this comment

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

Talked to Astro folks. They'll add a new category sometime next week. For now, we remove analytics. Can add the new category later on.

Suggested change
"analytics",

@Lms24 Lms24 marked this pull request as ready for review October 12, 2023 12:51
@Lms24 Lms24 force-pushed the lms/astro-integration-keywords branch from 3d8a4a9 to 0ffe696 Compare October 13, 2023 08:28
@Lms24 Lms24 mentioned this pull request Oct 13, 2023
@Lms24 Lms24 self-assigned this Oct 13, 2023
@Lms24 Lms24 merged commit a8c8564 into develop Oct 13, 2023
46 checks passed
@Lms24 Lms24 deleted the lms/astro-integration-keywords branch October 13, 2023 08: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.

None yet

2 participants