Skip to content

Add bugs metadata to published packages#398

Open
tonghuaxingdsb wants to merge 2 commits into
DataDog:masterfrom
tonghuaxingdsb:codex/add-vite-plugin-bugs-metadata
Open

Add bugs metadata to published packages#398
tonghuaxingdsb wants to merge 2 commits into
DataDog:masterfrom
tonghuaxingdsb:codex/add-vite-plugin-bugs-metadata

Conversation

@tonghuaxingdsb

@tonghuaxingdsb tonghuaxingdsb commented Jun 8, 2026

Copy link
Copy Markdown

Summary

  • add standard npm bugs.url metadata for all published packages in this repository
  • point the issue tracker link at the public DataDog/build-plugins issues page

Covered packages:

  • @datadog/esbuild-plugin
  • @datadog/rollup-plugin
  • @datadog/rspack-plugin
  • @datadog/vite-plugin
  • @datadog/webpack-plugin

Verification

  • node -e "const fs=require('fs'); const path=require('path'); const dir='packages/published'; const out={}; for (const name of fs.readdirSync(dir)) { const file=path.join(dir,name,'package.json'); if (!fs.existsSync(file)) continue; const p=require('./'+file.replace(/\\/g,'/')); out[p.name]={homepage:p.homepage,bugs:p.bugs,repository:p.repository}; if(p.bugs?.url!=='https://github.com/DataDog/build-plugins/issues') process.exitCode=1; } console.log(JSON.stringify(out,null,2));"
  • git diff --check

@yoannmoinet yoannmoinet left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for your contribution.
This should be done for all the published packages in the repository.
Not just vite.

@tonghuaxingdsb tonghuaxingdsb changed the title Add Vite plugin bugs metadata Add bugs metadata to published packages Jun 8, 2026
@tonghuaxingdsb

Copy link
Copy Markdown
Author

Thanks for the review. I updated the PR to add bugs.url to all published packages in packages/published: esbuild, rollup, rspack, vite, and webpack. I also updated the PR description with the full validation command.

@yoannmoinet yoannmoinet left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for your contribution, this looks good.

@yoannmoinet

Copy link
Copy Markdown
Member

@tonghuaxingdsb you need to sign your commits, can't merge without it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants