Skip to content

Generated v1.0 models and request builders using Kiota - #36

Closed
github-actions[bot] wants to merge 1 commit into
devfrom
v1.0/pipelinebuild/71164
Closed

github-actions[bot] wants to merge 1 commit into
devfrom
v1.0/pipelinebuild/71164

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

This pull request was automatically created by the GitHub Action, create pull request.

The commit hash is 683ad12.

Important Check for unexpected deletions or changes in this PR.

  • Check that the changes match the changes in the captured commit for v1.0_metadata.xml

@rkodev

Ronald K (rkodev) commented Mar 29, 2022

Copy link
Copy Markdown
Contributor

Closing this PR. We will not be updating models this week

@@ -1,2 +1,2 @@
export * from './archiveRequestBody'
export * from './archiveResponse'
export * from './archiveRequestBody'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vincent Biret (@baywet) Nikitha Chettiar (@nikithauc) is it possible an unordered set/map/list is being used when generating export statements? Replacing it with an ordered data structure should prevent unnecessary diffs

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that with the latest changes the only ordering that's applied is inheritance ordering, but that's it.
The ordering logic probably an additional ThenBy(x => x.Name) to be able to deterministically generate those.

@baywet

Copy link
Copy Markdown
Member

Ronald K (@rkodev) what's the rationale behind not updating the sdk weekly? we're setup for it and we should use that preview time to build experience and get good at releasing weekly.

@baywet
Vincent Biret (baywet) deleted the v1.0/pipelinebuild/71164 branch March 29, 2022 12:41
@rkodev

Ronald K (rkodev) commented Mar 29, 2022

Copy link
Copy Markdown
Contributor

Vincent Biret (@baywet) skipping updating the model this week since we have some manually modified files here #29

This should be resolved by addressing this ticket #33

This is only for the current week

@baywet

Copy link
Copy Markdown
Member

ok thanks for the detailed explanation here!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants