fix(get-dependency-list): add glob to dependency list and match existing behaviour on Windows.

[neverendingqs]

Resolves https://github.com/dougmoscrop/serverless-plugin-include-dependencies/issues/74#issuecomment-1213377321, https://github.com/dougmoscrop/serverless-plugin-include-dependencies/issues/75.

• f6bb143a9a7eeaf096e070d9b9c392afb533ee1e - glob is used at:
  • https://github.com/dougmoscrop/serverless-plugin-include-dependencies/blob/0c0fc72dd235c2ea5a25d45cf43c8b1c7ad8af37/get-dependency-list.js#L9
  • https://github.com/dougmoscrop/serverless-plugin-include-dependencies/blob/0c0fc72dd235c2ea5a25d45cf43c8b1c7ad8af37/include-dependencies.js#L7
• b777583e1e96faed546f4e6e24b3e92d2ee81f3d - based on the code today and glob@8 notes, I think this matches pre-glob@8.0 behaviour

It's not explicitly spelled out in the Serverless documentation, but it looks like package.patterns should always be using /s anyway, so I don't think https://github.com/dougmoscrop/serverless-plugin-include-dependencies/blob/0c0fc72dd235c2ea5a25d45cf43c8b1c7ad8af37/include-dependencies.js#L59-L65 needs to be updated with the same change.

Open to feedback!

---

Quality notes:

• Functional:
  • Was missing all node_modules/ files in Serverless package before this change, and this change pulled it in
  • Unit tests pass (https://github.com/neverendingqs/serverless-plugin-include-dependencies/pull/2)
  • If interested, I can submit the CI commit as another PR as well (https://github.com/neverendingqs/serverless-plugin-include-dependencies/pull/1)
    • I couldn't get unit tests running locally 😅
• Performance:
  • N/A
• Cost:
  • N/A
• Security:
  • N/A
• DevOps:
  • N/A
• Docs:
  • N/A