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

Merge 209 add information on what analysts should be testing into main #219

Merged

Conversation

zarbspace
Copy link
Contributor

I added some text about what to test and why. Ready to merge into main subject to review.

@zarbspace zarbspace added the enhancement ✨ New feature or request, or improvements to existing chapters label Jan 8, 2025
@zarbspace zarbspace requested a review from sarahcollyer January 8, 2025 13:21
@zarbspace zarbspace linked an issue Jan 8, 2025 that may be closed by this pull request
@zarbspace zarbspace requested a review from ellie-o January 8, 2025 14:05
Copy link
Contributor

@sarahcollyer sarahcollyer left a comment

Choose a reason for hiding this comment

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

Very clear and concise, happy for this to be merged down.

@ellie-o
Copy link
Contributor

ellie-o commented Jan 9, 2025

Have read from a less technical perspective - all very clear and useful.

@ellie-o ellie-o merged commit ff7b929 into main Jan 9, 2025
1 check passed
@ellie-o ellie-o deleted the 209-add-information-on-what-analysts-should-be-testing branch January 9, 2025 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ✨ New feature or request, or improvements to existing chapters
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add information on what analysts should be testing
3 participants