Use p-map instead of promise-pool

[firecow]

@naweiss Have a look. See #943 for more information.

promise-pool package doesn't throw on promise rejections, and we want that to happen, so developers can see error messages.

[naweiss]

@firecow can you give a minimal example to check the issue against?

[firecow]

@firecow can you give a minimal example to check the issue against?

Good idea, we don't want this to come back 👍

[firecow]

@naweiss I've added a test to prevent regression 👍

[github-advanced-security]

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
1 Code Smell

89.3% Coverage
0.0% Duplication