cdnjs / packages

📦 Package configurations - The #1 free and open source CDN built to make life easier for developers.
https://cdnjs.com
MIT License
369 stars 354 forks source link

Automate verification of library file map #1

Open MattIPv4 opened 5 years ago

MattIPv4 commented 5 years ago

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.

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!](https://www.bountysource.com/issues/75915046-automate-verification-of-library-file-map?utm_campaign=plugin&utm_content=tracker%2F32893&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F32893&utm_medium=issues&utm_source=github).
xtuc commented 4 years ago

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