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

Readme update testing #307

Merged
merged 4 commits into from
Feb 27, 2025
Merged

Readme update testing #307

merged 4 commits into from
Feb 27, 2025

Conversation

miskopo
Copy link
Member

@miskopo miskopo commented Mar 11, 2024

Provide information about testing in README.md file

@miskopo miskopo added the WIP Work in Progress (do not merge) label Mar 11, 2024
@miskopo miskopo self-assigned this Mar 11, 2024
@miskopo miskopo force-pushed the readme_update_testing branch 2 times, most recently from f080153 to 8c1a7be Compare March 11, 2024 12:48
@miskopo
Copy link
Member Author

miskopo commented Mar 11, 2024

In case it doesn't render properly in the diff, this is what Warning box looks like
image

@miskopo miskopo linked an issue Mar 11, 2024 that may be closed by this pull request
Copy link

This PR has not received any attention in 60 days.

@github-actions github-actions bot added the stale This PR/issue is stale and will be closed label May 11, 2024
Copy link
Collaborator

@mreynolds389 mreynolds389 left a comment

Choose a reason for hiding this comment

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

LGTM, I know its stale but lets get this in.

@github-actions github-actions bot removed the stale This PR/issue is stale and will be closed label May 17, 2024
Copy link

This PR has not received any attention in 60 days.

@github-actions github-actions bot added the stale This PR/issue is stale and will be closed label Jul 16, 2024
@github-actions github-actions bot closed this Jul 23, 2024
@duzda duzda reopened this Feb 24, 2025
@duzda duzda removed WIP Work in Progress (do not merge) stale This PR/issue is stale and will be closed labels Feb 24, 2025
Also added description for the unit tests and a note explaining, which
to use.

Fixes: freeipa#210
Signed-off-by: David Hanina <[email protected]>
Signed-off-by: David Hanina <[email protected]>
@duzda duzda force-pushed the readme_update_testing branch from 8c1a7be to 1ef7185 Compare February 25, 2025 09:07
@duzda duzda requested a review from carma12 February 25, 2025 09:07
Signed-off-by: David Hanina <[email protected]>
@duzda duzda force-pushed the readme_update_testing branch from c774400 to ac0c8e1 Compare February 25, 2025 09:13
Copy link
Collaborator

@carma12 carma12 left a comment

Choose a reason for hiding this comment

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

LGTM.

@duzda duzda merged commit 80c2e08 into freeipa:main Feb 27, 2025
4 checks passed
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.

Update README.md file to contain information about testing
4 participants