preserve imports from stale files #90
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds support for automatically preserving
import
statements infunctions.ts
andapi.ts
filesWhy
This was required because if a user had a setup in which they were using functions from files other than
api.ts
and had@save
annotation on those functions, re-running the codegen (introspect with overwrite set totrue
) would preserve the function, but would lose its import statement. This meant that the user would have to re-write their import statements again, which a really bad experience.Note
import
statements are always preserved and organized. They don't need a@save
annotation.Review
Since this is a relatively large PR, it'll be easier to review it per commit