Skip to content

Conversation

@MIchaelMainer
Copy link
Collaborator

We haven't merged in a while which indicates that we may want to consider getting rid of either master or dev.

MIchaelMainer and others added 30 commits April 24, 2018 10:28
Added metadata for April 2018 client library updates.
We need to generate POST/PATCH/DELETE for (I)EntityWithReferenceRequest.
A few of the base template methods did not have toggles for collection property initialization, so I added them where appropriate but set them to true by default, so it should not affect existing generated code.
…te-method

Add DELETE/POST/PATCH to (I)EntityWithReferenceRequest
Merge pull request #132 from microsoftgraph/fixWithRefRequest

Fixes generation issue for reference request
Fixing templates for java model generated files import statement for …
…template generates ~200 files and helps improve memory footprint for java sdk.
… template generates ~400 files and helps improve memory footprint for java sdk
Fixing BaseMethodBody java template files to use import statements co…
Java template fix to reduce memory footprint - continued
…e generates ~450 files. After this change, java sdk builds with 3.5GB RAM
…te generates ~450 files. After this change, java sdk with generated files builds with 3GB RAM.
Java template fix to reduce memory footprint - continued
Java template fix to reduce memory footprint - continued
deepak2016
deepak2016 previously approved these changes Jan 24, 2019
FAREAST\vidadhee and others added 4 commits January 24, 2019 18:43
#1. Add MSDuration for edm.duration handling.
#2. Change setters of Other date and time models
Sanitize Parameter variables to fix errors
@darrelmiller
Copy link
Contributor

We should discuss in triage tomorrow why we have dev and master. It's not clear to me.

darrelmiller
darrelmiller previously approved these changes Jan 30, 2019
JaapSingh and others added 4 commits January 29, 2019 22:24
Now allows GetSanitizedPropertyName to accept an additional parameter, suffix, so that an additional string may be concatenated onto the end of the Property. Updated method logic to look for and concatenate suffix when specified.
Updates IEntityRequestBuilder.Base.template.tt so that it does not create properties with the same name as the methods within the class (specifically request), by concatenating a "_" to the end of the property name.
Updates EntityRequestBuilder.Base.template.tt so that it does not create properties with the same name as the methods within the class (specifically request), by concatenating a "_" to the end of the property name.
JaapSingh and others added 6 commits February 1, 2019 09:04
Deletes invalid reference paths to avoid namespace errors
Fix build errors for same method and property name
Validated that this does not affect existing generation for .Net classes.
…GraphClientModels file and relevant imports
* Added a test for validating custom PHP namespace property from the TypeWriter CLI. Fixed test metadata.
* Add support for specifying key-value properties from the Typewriter CLI.
* Fix broken tests.
* Updated test metadata to match preprocessing
* Changes support E2E testing when different target platforms are used in a test run for Typewriter. The TemplateProcessor host needs to be reset for the case when we target a new language. Otherwise the wrong *CodeWriter will be used and an unsupported cast will occur in the templates when trying to cast host.CodeWriter to the platform specific CodeWriter. The change to ConfigurationService.Settings is also required to reset the settings for configuring the TemplateWriter.
* Updated typewriter tests to be more descriptive
Change Models template to ClientModels for ObjC
@MIchaelMainer MIchaelMainer merged commit df5eb92 into master Feb 12, 2019
MIchaelMainer added a commit that referenced this pull request Feb 12, 2019
Merge pull request #171 from microsoftgraph/dev
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants