Skip to content

Contribute TypeScript project detection to Smart Importer #54

Closed

Description

@angelozerr

Smart Importer is a very cool features to import project and discover while importing some files to add directly the well nature, etc

Today I have not TypeScript nature, but I think more and more we should have this TypeScript nature to hide for instance TypeScript from Project properties.

It should be cool when we import a project

  • it check that it exists a tsconfig.json or a ts files and add TypeScript nature to the project.
  • the buildpath is updated with directory folder which contains tsconfig.json

See sample with JPA at https://bugs.eclipse.org/bugs/show_bug.cgi?id=492323

@mickaelistria could you give me more information about "Smart Importer":

  • if I have understood, it will work only with Neon?
  • can I download none release of Neon to benefit with Smart Importer?
  • if each project uses RecursiveFileFinder to discover a file, is not a problem with performance? If we have JPA and TypeScript which provides smart importer, it means that files of project is visited twice?

Activity

  1. mickaelistria commented on Apr 26, 2016

    @mickaelistria
    Contributor

    I would recommend that as much as you can avoid a nature, do it. Natures introduce a lot of complexity both from maintenance and usage perspective.

    About the Smart Importer, Neon M6 has it. However, the API is subject to change. Using the recursiveFileFinder can cause performance issue.

  2. angelozerr commented on Apr 28, 2016

    @angelozerr
    OwnerAuthor

    I would recommend that as much as you can avoid a nature, do it.

    Many thanks for your recommandation! Today there are none nature. I use tsconfig.json to know if project is a typescript project or not. See my current work at #53 (comment)

  3. angelozerr commented on Apr 29, 2016

    @angelozerr
    OwnerAuthor

    @mickaelistria see https://github.com/angelozerr/typescript.java/wiki/TypeScript-BuildPath#typescript-nature if you are interested to know the rules how I manage "TypeScript Nature" without "Eclipse Nature".

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions