Skip to content

Add testing guide #46

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 65 commits into from
Feb 21, 2022
Merged

Add testing guide #46

merged 65 commits into from
Feb 21, 2022

Conversation

akaszynski
Copy link
Contributor

@akaszynski akaszynski commented Feb 11, 2022 •
edited
Loading

Initial attempt at adding testing section.

Will need to merge with new guidelines layout.

Resolves #19.

PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 11, 2022
View reviewed changes
@akaszynski
Copy link
Contributor Author

Pinging @jorgepiloto, please fix https://github.com/pyansys/dev-guide/runs/5187563828?check_suite_focus=true

@MaxJPRey
Copy link
Contributor

MaxJPRey commented Feb 15, 2022 •
edited
Loading

Pinging @jorgepiloto, please fix https://github.com/pyansys/dev-guide/runs/5187563828?check_suite_focus=true

@akaszynski, @jorgepiloto It is done, I just finished.

jorgepiloto
jorgepiloto reviewed Feb 17, 2022
View reviewed changes
seanpearsonuk
seanpearsonuk reviewed Feb 17, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 17, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 17, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 17, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 17, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 17, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 17, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 17, 2022
View reviewed changes
PipKat
PipKat reviewed Feb 17, 2022
View reviewed changes
MaxJPRey and others added 11 commits February 18, 2022 12:19
@MaxJPRey @PipKat
Update doc/source/guidelines/test_practices.rst
63ae117 
Co-authored-by: Kathy Pippert <[email protected]>
@MaxJPRey @PipKat
Update doc/source/guidelines/test_practices.rst
7348521 
Co-authored-by: Kathy Pippert <[email protected]>
@MaxJPRey @PipKat
Update doc/source/guidelines/test_practices.rst
92bf4db 
Co-authored-by: Kathy Pippert <[email protected]>
@MaxJPRey @PipKat
Update doc/source/guidelines/test_practices.rst
6dcbf9e 
Co-authored-by: Kathy Pippert <[email protected]>
@MaxJPRey @PipKat
Update doc/source/guidelines/test_practices.rst
96ad9a5 
Co-authored-by: Kathy Pippert <[email protected]>
@MaxJPRey @PipKat
Update doc/source/guidelines/test_practices.rst
7663139 
Co-authored-by: Kathy Pippert <[email protected]>
@MaxJPRey @PipKat
Update doc/source/guidelines/test_practices.rst
64d0207 
Co-authored-by: Kathy Pippert <[email protected]>
@MaxJPRey
End the sentence for the requirement files.
5706bf5
@akaszynski
add testing methodology
953ce59
@akaszynski
merge upstream
104cc3f
@MaxJPRey
Title underline was too short and prevented the doc to be buit.
14c1915
@akaszynski akaszynski merged commit 4360d52 into main Feb 21, 2022
@akaszynski akaszynski deleted the docs/add_doc_dep branch February 21, 2022 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jorgepiloto jorgepiloto jorgepiloto left review comments

@PipKat PipKat PipKat approved these changes

@MaxJPRey MaxJPRey MaxJPRey approved these changes

+1 more reviewer

@seanpearsonuk seanpearsonuk seanpearsonuk left review comments

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add testing section
5 participants
@akaszynski @MaxJPRey @jorgepiloto @PipKat @seanpearsonuk