redux connect can also be used with a decorator. But the decorator is placed above the class.

@vramana good call! It seems like redux connect takes arguments. We likely don't want decorators with arguments to be inlined. Let me encode this rule as well!

Done, now it has to be a single decorator AND just an identifier to be inlined.

I also tried to enable flow decorator support but it only supports half of the decorators in the tests and has a different AST structure, so it's not worth using it.

Thanks, looks good!

Is there a possibility to make this configurable? I personally really dislike decorators being in the same line. Makes it very hard to browse the class properties.

  @Entity("Todos")
  export class Todo implements ITodo {
    @PrimaryGeneratedColumn("uuid") public id?: string;
    @Column() public text: string;
    @Column() public completed: boolean;
  }

@jlongster @vjeux @vramana I agree highly with @egeozcan -- would love to see this be a configurable value. If I submit a PR, would the team be up for merging it or is everyone set on keeping things as is?

+1. This is very annoying, especially with Inversify.

public constructor(
    @inject(MyStoreStore) public readonly myStore: MyStore,
    @inject(InjectionTypes.StyleProvider) public readonly styleProvider: IStyleProvider,
) { }

Relevant issue: #1974

@JoshuaKGoldberg Please open a new issue, filling the template