Has the default configuration of scope-case changed?

[lxow456]

 When the contents of the commitlint.config.js are as follows:

module.exports = {
    extends: ['@commitlint/config-conventional']
}

 I ran this command

git commit -m "fix(SCOPE): fix"

Expected Behavior

 According to the content here: https://commitlint.js.org/#/reference-rules, the scope-case default value is lowerCase. This commit should be aborted.

Current Behavior

 But in fact, this message passed the verification. And here are my commit logs:

Possible Solution

 I added the configuration of the rules, as follows:

module.exports = {
    extends: ['@commitlint/config-conventional'],
    rules: {
        'scope-case': [2, 'always', ['lower-case']]
    }
} 

 Then I commit with a same message, this time it's aborted.

Your Environment

Executable	Version
@commitlint/config-conventional --version	16.2.4
@commitlint/cli --version	16.2.4
git --version	2.28.0.windows.1
node --version	14.7.0

[escapedcat]

Yeah, looks like it. Have a look at:

• Allow any scope casing for Conventional Commits #2529
• Allow any scope casing for Conventional Commits #828

I guess the README needs to be updated. Wanna create a PR for that?
Your solution looks good.

[lxow456]

@escapedcat Sure, my pleasure. I referenced to the #828. The pr modified @commitlint/config-conventional/index.js. Verification of scope-case has been removed. So I've come up with a modified scheme: Maybe we can delete the description of scope-case in @commitlint/config-conventional/README.md to update it.
 If you think this is feasible, I will raise the pr. Or, do you think there is any need to optimize this scheme? I am also happy to raise after optimization.😁

[lxow456]

@escapedcat In addition, I found that the docs/reference-rules.md also contains the description of scope-case configuration. In my understanding, the description here is from a global perspective. But the default values of scope-case are different in @commitlint/config-conventional/index.js and @commitlint/config-angular/index.js. So for the modification here, I have no feasible idea for the time being. I don't know if this is a problem. Just wanted to tell you about it~

[escapedcat]

Sorry for the late reply.

If you think this is feasible, I will raise the pr.

Let's start with this. I think it would be good clear this up.

But the default values of scope-case are different in @commitlint/config-conventional/index.js and @commitlint/config-angular/index.js.

I believe this is ok. conventional commits doens't define the scope case. Maybe angular does? I don't know tbh. But at least it's ok that they are not the same.