Configuration de linting partageable dans CI, quelque chose manque-t-il dans les vérifications ?

merefield · Mars 27, 2025, 7:08

Ceci passe dans une PR, je m’attendrais à ce que cela échoue à un lint ? :

github.com/merefield/discourse-tc-character-count

javascripts/discourse/connectors/after-composer-title-input/show-counts.gjs

95e47348a


      
              return this.args.outletArgs.composer.archetypeId === "private_message"
                ? this.siteSettings.min_personal_message_title_length
                : this.siteSettings.min_topic_title_length;
            }
          
            get missingTitleCharacters() {
              return this.titleRequiredLength - this.args.outletArgs.composer.titleLength;
            }
          
            <template>
              {{log this.args.outletArgs.composer}}
              <CharacterCounts
                @missingReplyCharacters={{this.missingTitleCharacters}}
                @length={{@outletArgs.composer.titleLength}}
                @minimumLength={{this.titleRequiredLength}}
              />
            </template>
          }

Les modèles gjs dans les fichiers de connecteur sont-ils ignorés ?

Je suis (euh) à peu près sûr d’utiliser une configuration actuelle :

github.com/merefield/discourse-tc-character-count

package.json

95e47348a


      
          {
            "private": true,
            "devDependencies": {
              "@discourse/lint-configs": "2.12.0",
              "ember-template-lint": "7.0.1",
              "eslint": "9.22.0",
              "prettier": "3.5.3",
              "prettier-plugin-ember-template-tag": "^2.0.4",
              "stylelint": "16.16.0"
            },
            "engines": {
              "node": ">= 18",
              "npm": "please-use-pnpm",

merefield · Mars 27, 2025, 7:45

Mes excuses.

C’était aussi simple qu’un point manquant sur le nom de fichier .template-lintrc.cjs

Envisagez d’utiliser GitHub - discourse/discourse-theme-skeleton: Template for Discourse themes lors de la configuration de nouveaux composants de thème… (bien que pour celui-ci, le linting ait été acquis par droit acquis)

Sujet		Réponses	Vues
Linting configuration Dev development	5	143	Septembre 11, 2024
Automatically updating themes and plugins to .gjs file format Dev dev-news	6	219	Mai 30, 2025
Suggestion: enable `eol-last` rule for eslint and ember-template-lint Dev	2	880	Avril 12, 2021
Is there a way to lint handlebars? Dev	14	2454	Février 18, 2021
Theme lint fails because -- Could not find "@discourse/lint-configs/stylelint" Dev	4	60	Mars 14, 2025

Configuration de linting partageable dans CI, quelque chose manque-t-il dans les vérifications ?

Sujets connexes