fix(package.json): replaced globby with glob #2370

[MadhaviMandalia]

Description

• Replaced globby version to glob in package.json

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.

[escapedcat]

Thanks for giving this a try!

Current status:

https://app.circleci.com/pipelines/github/conventional-changelog/commitlint/6332/workflows/1f3545a0-e863-4e6f-af0a-07f0f09c19fd/jobs/24378

The end result should be that all current tests are passing and linting works.

Try the following things:

1. Do a yarn, after that also the yarn.lock`-file should be updated.
2. Search for globby over the whole project and replace it with the new package
   i.e. places like this (and similar all other places):
   • https://github.com/MadhaviMandalia/commitlint/blob/master/%40commitlint/config-lerna-scopes/index.js#L4
   • https://github.com/MadhaviMandalia/commitlint/blob/master/%40commitlint/config-lerna-scopes/index.js#L24
   • Hopefully glob can be used exactly like globby. If the test fail, have a look at the readme

[MadhaviMandalia]

Ok thanks @escapedcat I will check and update.

[escapedcat]

Later on please modify your commit to i.e.:
chore: replaced globby with glob #2370
Because this won't change the code, no need to use fix, feat or refactor. chore is fine.

You can change the commit by using i.e. git commit --amend and doing git push --force after.

[MadhaviMandalia]

I don't think we should change this because that would call local function glob. Please let me know if there is any change.

https://github.com/MadhaviMandalia/commitlint/blob/35ca695ae49df71fa636171cba2547aefc4a7357/%40commitlint/rules/src/index.test.ts#L31

[escapedcat]

I don't think we should change this because that would call local function glob. Please let me know if there is any change.

Yeah, not only the package is being switched, also the function has to be switched accordingly everywhere were currently globby is being used.
Have a look at how glob should be used: https://github.com/isaacs/node-glob#usage

[escapedcat]

@MadhaviMandalia let me know if you need help with this

[escapedcat]

Closing this for now. Might re-open when we continue.