Skip to content

fix: correct misspelled @deprecated TSDoc tag on UseJsPropertyNames#9314

Open
SSDWGG wants to merge 1 commit into
iTwin:masterfrom
SSDWGG:fix/misspelled-deprecated-tag
Open

fix: correct misspelled @deprecated TSDoc tag on UseJsPropertyNames#9314
SSDWGG wants to merge 1 commit into
iTwin:masterfrom
SSDWGG:fix/misspelled-deprecated-tag

Conversation

@SSDWGG
Copy link
Copy Markdown

@SSDWGG SSDWGG commented May 16, 2026

Summary

Fixes #9294 — the @deprecated TSDoc tag on QueryRowFormat.UseJsPropertyNames was misspelled as @depreacted, preventing IDE deprecation warnings and ESLint rules from flagging the deprecated API.

Change

core/common/src/ConcurrentQuery.ts line 30:

- * @depreacted in 4.11.  Switch to UseECSqlPropertyIndexes...
+ * @deprecated in 4.11.  Switch to UseECSqlPropertyIndexes...

Impact

After this fix, TypeScript compilers, VS Code, and ESLint @typescript-eslint/no-deprecated rules will correctly flag usage of QueryRowFormat.UseJsPropertyNames as deprecated.

Type of Change

  • Bug fix (TSDoc correction)

The deprecation tag on QueryRowFormat.UseJsPropertyNames was spelled
@depreacted instead of @deprecated, preventing TypeScript compilers,
ESLint rules, and IDE tooling from flagging the deprecated API.

Fixes iTwin#9294
@SSDWGG SSDWGG requested review from a team as code owners May 16, 2026 02:05
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 16, 2026

CLA assistant check
All committers have signed the CLA.

@hl662
Copy link
Copy Markdown
Contributor

hl662 commented May 16, 2026

Thanks for opening a PR contribution! We already have a PR drafted by another contributor that resolves this issue: #9295

The steps to correctly resolve this issue is a lot trickier than fixing a typo, if you are have some time, could you use the PR linked above to help you get your PR to the finish line?

@markschlosseratbentley markschlosseratbentley dismissed their stale review May 18, 2026 11:39

Comment from hl662.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix misspelled @deprecated tag on QueryRowFormat.UseJsPropertyNames

4 participants