search

tfso / eslint-config-tfso

EsLint configuration for 24SevenOffice
0 stars 0 forks source link

Fungerer extends med nytt pakkenavn? #5

Open lostfields opened 3 years ago

lostfields commented 3 years ago

Med endring på pakkenavn, vil gamle pakkereferanse i ESlint config fungere da?

https://github.com/tfso/eslint-config-tfso/blob/7eb065b8c47970fefe067fd1db7ebe98137c384e/package.json#L2

Kort referanse som fungerer fordi pakkenavn var eslint-config-tfso

{
  "extends": [
    "tfso/typescript"
  ]
}
OysteinVesth commented 3 years ago

Good catch @lostfields, var faktisk ikke klar over at pakka var public, så var unødvendig av meg uansett. Bytter tilbake navn og fjerner deployment av pakke, men beholder tags.

OysteinVesth commented 3 years ago

Renamet til gammelt navn, men beholder changelog GH action

lostfields commented 3 years ago

Det vil fungere iht dokumentasjonen på https://eslint.org/docs/developer-guide/shareable-configs#npm-scoped-modules men man må endre extend fra tfso til @tfso. "Problemet" kommer når man skal spesifisere en spesiell config i prosjektet, f.eks. tfso/react, og med scoped navn så må man bruke @tfso/eslint-config/react istedenfor.

Selv om vi endrer navn på package.json så vil vel det gamle fungere, for det er jo navnet på pakkareferansen som bestemmer navnet på mappen under node_modules.

Bare kjøre på slik det var @OysteinVesth ?