deprecation syntax inconsistent for objects, not possible for const/let/var

[timotheecour]

• {.deprecated: "msg" .} works for routines/types but not for const/let/var:

block:
  # works
  const Foo {.deprecated.}  = 1

block:
  # Error: annotation to deprecated not supported here
  # const Foo {.deprecated: "bla".}  = 1

• deprecated for object type inconsistent all other deprecated forms:
  for pretty much everything, {.deprecated: "blabla".} goes before the = but not for object type:

block:
  # object type: deprecated goes after `=`
  type Foo = object {.deprecated: "blabla".}
    a: int

block:
  # every thing else: deprecated goes before `=`: enum, routines, const/let/var
  type Foo {.deprecated: "asdf" .} = enum
    a

[alaviss]

This could have been separated to two issues, each with a small, not an essay title:

• deprecated pragma with annotation doesn't work for variables
• deprecated pragma is inconsistant for object type

Seriously, you should start minimizing your issue title to not scare people away, avoiding symbols might be a good start. Just my 2¢.

[timotheecour]

Seriously, you should start minimizing your issue title to not scare people away, avoiding symbols might be a good start. Just my 2¢.

I understand your points and will keep in mind; there's a tradeoff between precision and conciseness; regarding symbols, {.deprecated: "msg" .} may make it easier to search (via {.deprecated: ") compared to deprecated pragma with annotation (lots of synonyms in english) when searching with in:title but YMMV
likewise with grouping vs splitting related issues; in some similar past cases others were recommending grouping instead; so it's not completely obvious.

[nc-x]

Part 2 fixed by - 05b8085
Part 1 - PR #10110 These symbols do not store enough information currently so the patch does not work (more information). Patch added for vars/lets #10234
Only remaining issue is const