Skip to content

Conversation

PipKat
Copy link
Member

@PipKat PipKat commented Jul 21, 2022

PyAnsys libraries are to follow the Google developer's style guide, which uses sentence case for headings. It also recommends active voice, present tense and short, clear sentences. Once Vale is implemented in the Fluent repos, it will catch all violations of the Google guidelines. Vale has been run on this FR and all violations are fixed. New contributions must pass the Vale style check.

@dnwillia-work dnwillia-work requested a review from ypatel-qa July 22, 2022 14:54
@dnwillia-work dnwillia-work requested a review from smf2001 July 22, 2022 15:17
@ansys ansys deleted a comment from dnwillia-work Jul 22, 2022
@dnwillia-work
Copy link
Collaborator

Would someone be able to help me get the Build and Unit Testing check to pass. I have trouble deciphering the details on this! Given the size, I'd assume we'd want to get this finalized and merged ASAP. Thanks!

Now the latest one did not pull the docker image. Not sure why. We did add a 23.1 image today, let me check....

@dnwillia-work
Copy link
Collaborator

Would someone be able to help me get the Build and Unit Testing check to pass. I have trouble deciphering the details on this! Given the size, I'd assume we'd want to get this finalized and merged ASAP. Thanks!

It looks ok now after a few retries.

Incorporate reviewer comments

Co-authored-by: Alex Kaszynski <akascap@gmail.com>
Attempt to fix failing style check
@PipKat PipKat merged commit 61aa2e9 into main Jul 25, 2022
@PipKat PipKat deleted the doc/overall_edit_for_PyAnsys_style branch July 25, 2022 18:40
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.

7 participants