packages icon indicating copy to clipboard operation
packages copied to clipboard

Automate verification of library file map

Open MattIPv4 opened this issue 6 years ago • 1 comments
trafficstars

Idea

Currently, when maintainers go to manually review a pull request to add a library, they have to confirm with the PR author that any files missed by the file map are intentionally missed. Additionally, it has to be confirmed that all files falling under the map are also intentionally included.

To save time and automate this process, it would be great if @PeterBot automatically listed files to the author that are not included in the file map as well as the files that the map does pick up to confirm it is intended.

  • Make the PR author aware of files missed by the file map and confirm this is intended
  • Make the PR author aware of files picked up by the map and confirm intention

Relevant Scripts

CI Script: https://github.com/cdnjs/cdnjs-drone-ci/blob/master/cdnjs.sh PR Review Script: https://github.com/PeterDaveHello/Drone-GitHub-status-report-bot/blob/master/main.sh


Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

MattIPv4 avatar Jun 22 '19 10:06 MattIPv4

That's a good idea, it could be part of the future linting of cdnjs/packages

xtuc avatar Feb 20 '20 00:02 xtuc