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

Implementing new scenarios: wait-pin and wait-pin-change #12

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

rafaelcorsi
Copy link

@rafaelcorsi rafaelcorsi commented Feb 2, 2024

Hi there!

Contributing in favor of fix #6.

I think that in some cases, it would be useful if we added a timeout specification for each of the wait commands.

@urish
Copy link
Contributor

urish commented Feb 3, 2024

Thanks! I plan to review it over the next few days and provide feedback.

@urish urish self-requested a review February 3, 2024 16:35
@urish urish closed this Feb 5, 2024
@urish urish reopened this Feb 5, 2024
@urish
Copy link
Contributor

urish commented Feb 12, 2024

Sorry, haven't gotten around to this yet - but it's still on my list

@urish urish self-assigned this Feb 12, 2024
@urish urish added the enhancement New feature or request label Feb 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New scenarios for CI/CD - wait on pin value
2 participants