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

[typespec-python] typespec-python emitter sphinx/linting #2478

Open
l0lawrence opened this issue Mar 29, 2024 · 4 comments
Open

[typespec-python] typespec-python emitter sphinx/linting #2478

l0lawrence opened this issue Mar 29, 2024 · 4 comments

Comments

@l0lawrence
Copy link
Member

On emitter 0.22.5 version runs I am seeing trailing whitespace warnings in Pylint in generated files, and sphinx warnings

@l0lawrence
Copy link
Member Author

the sphinx error is coming from cpython - not an emitter issue :) @kristapratico

@iscai-msft
Copy link
Contributor

sweet, can i close this issue then?

@kristapratico
Copy link
Member

@iscai-msft let's not worry about the sphinx warning. @l0lawrence can you double-check if the trailing whitespace warnings are coming from black not running after generation?

@l0lawrence
Copy link
Member Author

Sorry for the late update, it does look like it is not running black because it fails but it does generate code --

ValueError: Something happened with tsp-client update step: CompletedProcess(args='tsp-client update', returncode=1, stderr=b'Traceback (most recent call last):\r\n File "C:\Users\llawrence\AppData\Local\Programs\Python\Python310\lib\runpy.py", line 196, in _run_module_as_main\r\n return _run_code(code, main_globals, None,\r\n File "C:\Users\llawrence\AppData\Local\Programs\Python\Python310\lib\runpy.py", line 86, in _run_code\r\n exec(code, run_globals)\r\n File "C:\Users\llawrence\Repositories\azure-sdk-for-python\sdk\eventgrid\azure-eventgrid\TempTypeSpecFiles\Azure.Messaging.EventGrid\node_modules\@AutoREST\python\autorest\black\init.py",

I'm not sure if this is EG typespec related or not

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

No branches or pull requests

3 participants