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

Make preamp tests find aptester #732

Closed
wants to merge 0 commits into from
Closed

Conversation

rtertiaer
Copy link
Contributor

@rtertiaer rtertiaer commented May 22, 2024

What does this change intend to accomplish?

For some reason aptester commands need a bigger timeout. This puts a bandaid on it without fixing the underlying problem.

Checklist

  • Have you tested your changes and ensured they work? (Lincoln did)

@rtertiaer rtertiaer requested a review from linknum23 May 22, 2024 17:33
@linknum23 linknum23 changed the title Lincoln made testing changes ad-hoc, and surfaced them during a IRL chat Make preamp tests find aptester May 22, 2024
Copy link
Contributor

@linknum23 linknum23 left a comment

Choose a reason for hiding this comment

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

🥇

@rtertiaer rtertiaer closed this May 22, 2024
@Lohrer Lohrer deleted the lincoln_changes_irl branch May 28, 2024 14:34
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.

None yet

2 participants