Skip to content

Conversation

@iscai-msft
Copy link
Contributor

@iscai-msft iscai-msft commented Dec 8, 2020

fixes #841

added tests based off of @jsntcy request, see this merged pr for the test case I added.

As proof sphinx renders it the same whether an enum comment is on the same line or not, here's rendering of before and after this fix
before

image

after:
image

cc @00Kai0

@iscai-msft iscai-msft requested a review from lmazuel December 8, 2020 22:56
@iscai-msft iscai-msft merged commit bfc14a5 into autorestv3 Dec 15, 2020
@iscai-msft iscai-msft deleted the comment_bug branch December 15, 2020 23:12
iscai-msft added a commit that referenced this pull request Dec 23, 2020
…into paging_separate_operations

* 'autorestv3' of https://github.com/Azure/autorest.python:
  add enum description son top of declaration, wrap comments (#844)
  add default value 'api-key' for `--credential-key-header-name` (#840)
  update head-as-boolean spec
iscai-msft added a commit that referenced this pull request Jan 4, 2021
…into improve_lro_docstrings

* 'autorestv3' of https://github.com/Azure/autorest.python:
  add enum description son top of declaration, wrap comments (#844)
  add default value 'api-key' for `--credential-key-header-name` (#840)
  update head-as-boolean spec
iscai-msft added a commit that referenced this pull request Jan 5, 2021
…into black_plugin

* 'autorestv3' of https://github.com/Azure/autorest.python:
  fix default polling method listed in docstring (#830)
  add enum description son top of declaration, wrap comments (#844)
  add default value 'api-key' for `--credential-key-header-name` (#840)
  update head-as-boolean spec
iscai-msft added a commit that referenced this pull request Jan 7, 2021
…into docs

* 'autorestv3' of https://github.com/Azure/autorest.python:
  add black flag (#836)
  fix default polling method listed in docstring (#830)
  add enum description son top of declaration, wrap comments (#844)
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.

Codegen bug caused by line wrapping of comment statements

3 participants