Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

readability-identifier-naming NamespaceAliasCase feature request #109385

Open
Fiero99 opened this issue Sep 20, 2024 · 1 comment
Open

readability-identifier-naming NamespaceAliasCase feature request #109385

Fiero99 opened this issue Sep 20, 2024 · 1 comment
Labels
clang-tidy enhancement Improving things as opposed to bug fixing, e.g. new or missing feature

Comments

@Fiero99
Copy link

Fiero99 commented Sep 20, 2024

Feature request:

check identifier naming on alias namespaces

Example:

namespace NamespaceAlias = some_namespace; // Warning NamespaceAlias should be namespace_alias

What I already tried:

With .clang-tidy

  - key:             readability-identifier-naming.NamespaceCase 
    value:           lower_case

The following alias namespace is not checked.

namespace some_namespace {}
namespace NamespaceAlias = some_namespace;

I would expect the alias namespace to be in the same format as other namespaces. (Or to be able to configure it using something like NamespaceAliasCase.

@EugeneZelenko EugeneZelenko added enhancement Improving things as opposed to bug fixing, e.g. new or missing feature clang-tidy and removed new issue labels Sep 20, 2024
@llvmbot
Copy link
Collaborator

llvmbot commented Sep 20, 2024

@llvm/issue-subscribers-clang-tidy

Author: None (Fiero99)

**Feature request:**

check identifier naming on alias namespaces

Example:

namespace NamespaceAlias = some_namespace; // Warning NamespaceAlias should be namespace_alias

What I already tried:

With .clang-tidy

  - key:             readability-identifier-naming.NamespaceCase 
    value:           lower_case

The following alias namespace is not checked.

namespace some_namespace {}
namespace NamespaceAlias = some_namespace;

I would expect the alias namespace to be in the same format as other namespaces. (Or to be able to configure it using something like NamespaceAliasCase.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clang-tidy enhancement Improving things as opposed to bug fixing, e.g. new or missing feature
Projects
None yet
Development

No branches or pull requests

4 participants
@Fiero99 @EugeneZelenko @llvmbot and others