[BUG] Linter is_snake_case only matches ASCII — always fails for Devanagari identifiers
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 56/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- Calme
- Stack technique
- python
- Domaine
- compilers, internationalization
Piste de recherche
The naming rule is is_snake_case in maithili_dsl/transpiler/linter.py, and its ASCII-only regex is the cause. Check where enforce_snake_case in LINT_CONFIG is read, and run 'नाम'.isidentifier() to confirm the identifier is valid. Done when Devanagari names pass the check, Latin snake_case is still enforced, and new cases in tests/test_linter.py cover both. The issue leaves the script-detection approach open, so confirm it with a maintainer before choosing one.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
File: maithili_dsl/transpiler/linter.py:11
def is_snake_case(name):
return bool(re.match(r'^[a-z_][a-z0-9_]*$', name))
The regex only matches ASCII lowercase, so every Devanagari variable name (e.g. नाम, गिनती) fails the check. Devanagari identifiers are valid Python 3 identifiers ('नाम'.isidentifier() is True), so the naming rule should accept them.
Currently mitigated because enforce_snake_case defaults to False in LINT_CONFIG, but any user who enables it gets a false positive on every line.
Fix: make the naming check script-aware — e.g. accept identifiers composed of Devanagari characters (and underscores/digits), and only apply the ASCII snake_case rule to Latin-script names. Add cases to tests/test_linter.py.
From BACKLOG.md item 10.
- Langage dominant
- Python
- Étoiles
- 1
- Forks
- 4
- Merge moyen
- 12 j 21 h
- PR mergées (30 j)
- 1
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de alphacrack/python-maithili-dsl
-
help wanted infra P2 testing
Difficulté 2/5 1-3 heures Accessibilité débutants 80/100
-
documentation enhancement good first issue help wanted P3
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
documentation good first issue P3
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
-
area:linter enhancement good first issue P3 translation
Difficulté 2/5 Une demi-journée Accessibilité débutants 66/100
-
area:cli enhancement good first issue P3
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
Toutes les issues de alphacrack/python-maithili-dsl
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
gorakhargosh/watchdog#1462 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
openwisp/openwisp-monitoring#889 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
OpenFreeEnergy/Lomap#180 ·
-
generate_observation_scenario_from_cpgs exposure time conversion when auto-gain set to 'yes'Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
roman-corgi/corgisim#327 ·
Les mainteneurs répondent en général sous 2 jours
-
agent:WSL bug code-review linux pri:medium sev:low tooling
Difficulté 2/5 Une demi-journée Accessibilité débutants 62/100
Les mainteneurs répondent en général sous 1 jour