Skip to content
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

Typescript - introducing index files namespace wise fixing circular dependency #1274

Merged
merged 34 commits into from
Mar 21, 2022

Conversation

Copy link
Member

@baywet baywet left a comment

Choose a reason for hiding this comment

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

Thanks for taking a go at it. On top of all the comments I've already made, would you mind adding a changelog entry please?

src/Kiota.Builder/GenerationConfiguration.cs Outdated Show resolved Hide resolved
src/Kiota.Builder/GenerationConfiguration.cs Outdated Show resolved Hide resolved
src/Kiota.Builder/Refiners/CommonLanguageRefiner.cs Outdated Show resolved Hide resolved
src/Kiota.Builder/CodeRenderer.cs Outdated Show resolved Hide resolved
@baywet baywet added the TypeScript Pull requests that update Javascript code label Feb 28, 2022
@baywet baywet added this to the Community Preview milestone Feb 28, 2022
@baywet
Copy link
Member

baywet commented Mar 2, 2022

@nikithauc FYI I just merged main into this branch to solve conflicts with all recent changes. I didn't rebase in case you have local uncomitted or unpushed changes to avoid making your life more complicated. Just remember to pull next time you work on this.

Copy link
Member

@baywet baywet left a comment

Choose a reason for hiding this comment

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

this is good progress, here are a few comments

src/Kiota.Builder/Writers/RelativeImportManager.cs Outdated Show resolved Hide resolved
src/Kiota.Builder/CodeRenderers/TypeScriptCodeRenderer.cs Outdated Show resolved Hide resolved
src/Kiota.Builder/CodeRenderers/CodeRenderer.cs Outdated Show resolved Hide resolved
src/Kiota.Builder/CodeRenderers/CodeRenderer.cs Outdated Show resolved Hide resolved
…erTests.cs

Co-authored-by: Mustafa Zengin <muzengin@microsoft.com>
baywet
baywet previously approved these changes Mar 18, 2022
Signed-off-by: Vincent Biret <vibiret@microsoft.com>
baywet
baywet previously approved these changes Mar 18, 2022
CHANGELOG.md Outdated Show resolved Hide resolved
baywet
baywet previously approved these changes Mar 21, 2022
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

85.9% 85.9% Coverage
0.0% 0.0% Duplication

@nikithauc nikithauc merged commit dbb3616 into main Mar 21, 2022
@nikithauc nikithauc deleted the nikithauc/test-index branch March 21, 2022 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TypeScript Pull requests that update Javascript code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug TypeScript: Runtime error when executing generated code for Graph API
3 participants