Last 12 weeks ยท 17 commits
2 of 6 standards met
This PR attempts to address the TS errors currently present in . I'm hoping to eventually add type checking to the CI job, but the first step towards that is addressing the existing errors. In I've just added some casts. I didn't see a simple alternative. The code in contained a type error, but it didn't seem that code was actually being used anywhere, so I've removed it. The output files generated in seem unaffected by removing that file. contained 2 type errors related to : 1. The property is included in the type but was missing from the object. While we don't use elsewhere, it's part of the type coming from v6. Based on it seemed easy enough to add it, so I just added it in. I doubt anyone is actually using it, but I think this technically counts as a , which is why I've titled this PR rather than . 2. The property is included in the object but missing from the type. This inconsistency was introduced by #421, which adjusted the type to match v6. There are various ways I could have fixed this, but I chose to add to the type. I marked it as optional for better consistency with the v6 types, though in practice we do always set the property in the current code.
Repository: vuejs/devtools. Description: โ๏ธ Browser devtools extension for debugging Vue.js applications. Stars: 2781, Forks: 242. Primary language: TypeScript. Languages: TypeScript (75.7%), Vue (22.5%), HTML (1%), CSS (0.6%), JavaScript (0.2%). License: MIT. Homepage: https://devtools.vuejs.org/ Latest release: v8.0.7 (5d ago). Open PRs: 32, open issues: 177. Last activity: 1h ago. Community health: 37%. Top contributors: webfansplz, alexzhang1030, renovate[bot], Azurewarth0920, skirtles-code, LoTwT, JianJroh, yyx990803, btea, xuzuodong and others.
TypeScript