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

[Doc] Avocado instrumented lifecycle #6012

Merged

Conversation

richtja
Copy link
Contributor

@richtja richtja commented Aug 27, 2024

This commit adds a new chapter to the reference guide about avocado instrumented tests and its lifecycle.

Reference: #5995

@mr-avocado
Copy link

mr-avocado bot commented Aug 27, 2024

Dear contributor,
Avocado is currently at the end of sprint 107, therefore we are in feature freeze state.
Please avoid merging changes that do not fall into these categories:

  • Bug fixes
  • Documentation updates

The feature freeze will be active until the release planned on September 02, 2024.

Copy link
Contributor

@clebergnu clebergnu left a comment

Choose a reason for hiding this comment

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

Hi @richtja,

The important things here seems to be in the right shape. Just a few wording suggestions and typos caught, though.

This commit adds a new chapter to the reference guide about avocado
instrumented tests and its lifecycle.

Reference: avocado-framework#5995
Signed-off-by: Jan Richter <[email protected]>
@richtja
Copy link
Contributor Author

richtja commented Sep 11, 2024

Hi @clebergnu, thanks for your review, the English grammar is a pain in the neck. I did fix via force-push. Please have a look.

Copy link
Contributor

@clebergnu clebergnu left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@clebergnu clebergnu merged commit 111d70a into avocado-framework:master Sep 11, 2024
58 checks passed
@richtja richtja deleted the instrumented_lifecycle_doc branch October 9, 2024 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done 108
Development

Successfully merging this pull request may close these issues.

2 participants