Shareable linting config in CI, is something missing in checks?

merefield · March 27, 2025, 7:08am

This is passing in a PR, I would expect it to fail a 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>
          }

Are gjs templates in connector files being missed?

Pretty (ahem) sure I’m using a current config:

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 · March 27, 2025, 7:45am

My bad.

It was as simple as a missing dot on the .template-lintrc.cjs filename

consider using the GitHub - discourse/discourse-theme-skeleton: Template for Discourse themes when setting up new theme components … (although for this one, linting was grandfathered)

Topic		Replies	Views
Composer Character Count Theme component	5	66	March 27, 2025
Character count when writing description Feature composer , characters-required	14	1752	March 24, 2025
Suggestion for the "Characters Required" plugin Feature	11	46	March 24, 2025
Travis tests failing due to eslint Dev	8	1112	September 12, 2020
Code-block line numbers Theme component code	18	404	March 3, 2025

Shareable linting config in CI, is something missing in checks?

Related topics