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

Run API stub generator using published package #3086

Conversation

praveenkuttappan
Copy link
Member

This PR has changes to optimize virtual env creation step to use APP-DATA and symlinks and also run api stub gen using package module.

@@ -17,29 +17,27 @@ public class PythonLanguageService : LanguageProcessor
{
public override string Name { get; } = "Python";
public override string Extension { get; } = ".whl";
public override string VersionString { get; } = "0.2.8";
public override string VersionString { get; } = "0.2.11";
Copy link
Member

Choose a reason for hiding this comment

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

What is this needed for (I can see I haven't updated it--oops)?

Copy link
Member Author

Choose a reason for hiding this comment

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

we already published 0.2.10 to feed. I need latest changes in this PR to integrate published package to APIVIew to run stubgen as a module. So only option is to publish a new version and refer the same when generating/ checking if update is required.

Copy link
Member Author

Choose a reason for hiding this comment

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

This property is still used to check if we need to auto update an existing review or not.

@praveenkuttappan praveenkuttappan self-assigned this Apr 6, 2022
@praveenkuttappan praveenkuttappan added the Central-EngSys This issue is owned by the Engineering System team. label Apr 6, 2022
@praveenkuttappan praveenkuttappan merged commit cdc135c into Azure:main Apr 6, 2022
@praveenkuttappan praveenkuttappan deleted the feature/install_published_python_parser branch April 6, 2022 21:14
ruowan pushed a commit to ruowan/azure-sdk-tools that referenced this pull request Apr 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Central-EngSys This issue is owned by the Engineering System team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants