dsherret / ts-type-info

TypeScript AST and code generator [Deprecated]
MIT License
94 stars 8 forks source link

Remove renameDefinitionAs #225

Closed dsherret closed 7 years ago

dsherret commented 7 years ago

The code for that is too complicated and #224 will allow for just doing:

def.name = "MyNewName";

I'm excited for that.

dsherret commented 7 years ago

I'm glad this code is gone. I'm sure there were problems with it. The new way will be much better once it's implemented.

dsherret commented 7 years ago

I'll be reverting this commit. Unfortunately, it was too complicated to implement a way for renaming definitions that's reliable. Plus, in ts-simple-ast it is extremely easy and reliable to rename definitions and people should start using that library for code manipulation soon.