Skip to content

doc: move testing section from HACKING to its own doc#739

Merged
OddBloke merged 3 commits into
canonical:masterfrom
OddBloke:doc
Dec 18, 2020
Merged

doc: move testing section from HACKING to its own doc#739
OddBloke merged 3 commits into
canonical:masterfrom
OddBloke:doc

Conversation

@OddBloke
Copy link
Copy Markdown
Collaborator

@OddBloke OddBloke commented Dec 18, 2020 •
edited
Loading

Proposed Commit Message

doc: move testing section from HACKING to its own doc

Our HACKING doc is very long, making it hard to justify expanding or
restructuring its content too much. This moves the testing section to
its own doc, and adds section headers in to make finding/linking to
testing guidelines easier.

To avoid confusion, the title of debugging.rst is changed.

Test Steps

tox -e doc; xdg-open doc/rtd_html/topics/hacking.html to click through to the new page.

Checklist:

  • My code follows the process laid out in the documentation
  • I have updated or added any unit tests accordingly
  • I have updated or added any documentation accordingly

ubuntu-server-builder
ubuntu-server-builder approved these changes Dec 18, 2020
View reviewed changes
Copy link
Copy Markdown
Collaborator

@ubuntu-server-builder ubuntu-server-builder left a comment

Choose a reason for hiding this comment

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

LGTM!

blackboxsw
blackboxsw approved these changes Dec 18, 2020
View reviewed changes
Copy link
Copy Markdown
Collaborator

@blackboxsw blackboxsw left a comment

Choose a reason for hiding this comment

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

LGTM non-bot Chad :)

@OddBloke OddBloke merged commit 5d017ab into canonical:master Dec 18, 2020
@OddBloke OddBloke deleted the doc branch December 18, 2020 20:07
This was referenced May 12, 2023
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@blackboxsw blackboxsw blackboxsw approved these changes

+1 more reviewer

@ubuntu-server-builder ubuntu-server-builder ubuntu-server-builder approved these changes

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.

3 participants

@OddBloke @blackboxsw @ubuntu-server-builder