forked from open-toast/protokt
-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove importresolver package #5
Merged
andrewparmet
merged 30 commits into
andrewparmet:kotlinpoet
from
pranjalvachaspati-toast:convert-to-types
Oct 21, 2021
Merged
Changes from all commits
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
7beadd7
Use 4 spaces instead of 2 for default indent
pranjalvachaspati-toast 5665e73
Remove message.stg
pranjalvachaspati-toast 2cc5b3a
Remove Descriptor template
pranjalvachaspati-toast 7ecc79f
Remove Descriptor template
pranjalvachaspati-toast 14bd064
Remove dead code in header template and most of services template.
pranjalvachaspati-toast 0861dcd
Set locale for decapitalize to get rid of compiler warning.
pranjalvachaspati-toast 1cca8d0
Remove unused templates
pranjalvachaspati-toast 151e25c
Create subpackage for annotators.
pranjalvachaspati-toast 945b990
Merge branch 'kotlinpoet' of https://github.com/andrewparmet/protokt …
pranjalvachaspati-toast e24fb1d
Move methodType to kotlinpoet and get rid of services.stg
pranjalvachaspati-toast 0369624
Remove Oneof stringtemplates.
pranjalvachaspati-toast 5964ab4
Get rid of a couple renderers
pranjalvachaspati-toast 46839ce
Migrate boxing & default value renderers away from StringTemplate.
pranjalvachaspati-toast f12ca3c
Remove sizeof stringtemplates.
pranjalvachaspati-toast 9981c53
Add non-stringtemplate serializer generation
pranjalvachaspati-toast ec526a1
Get rid of renderers stringtemplate
pranjalvachaspati-toast 91accd6
Remove options stringtemplate
pranjalvachaspati-toast 8aec02e
--amend
pranjalvachaspati-toast 8223782
Merge branch 'kotlinpoet' of https://github.com/andrewparmet/protokt …
pranjalvachaspati-toast 4f4d40a
Convert deserializer to use kotlinpoet
pranjalvachaspati-toast d4af5e5
Create packages for import resolution and descriptor stuff
pranjalvachaspati-toast f38ffed
Create packages for import resolution and descriptor stuff
pranjalvachaspati-toast 2125158
This merge required some additional changes - mostly changing Strings…
pranjalvachaspati-toast 9b264a3
Additional changes from Strings to CodeBlocks
pranjalvachaspati-toast 8bb76ce
Disable non-kotlinpoet import resolution
pranjalvachaspati-toast db9d992
Remove importresolution package
pranjalvachaspati-toast 764782c
Merge branch 'kotlinpoet' of https://github.com/andrewparmet/protokt …
pranjalvachaspati-toast 1f53989
Make fixes for PR comments
pranjalvachaspati-toast e84eae0
Use DSL typename in DeserializerAnnotator
pranjalvachaspati-toast a4193df
Remove dependency on stringtemplate
pranjalvachaspati-toast File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yikes. (I know this came straight from StringTemplate.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup, there are a few messy templates in here that should probably be cleaned up and kotlinpoet-ified.