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

Updated last 4 student code test files #277

Merged
merged 2 commits into from
Jun 12, 2024

Conversation

3142008956
Copy link
Member

Updated last 4 files in tests/student_code to account for the updated executor. Replaced autonomous setup/main with autonomous() and teleop setup/main with teleop(), and added while True loops to simulate continuous execution.

@3142008956 3142008956 requested a review from BrandonWong14 June 12, 2024 03:40
@3142008956 3142008956 closed this Jun 12, 2024
@3142008956 3142008956 reopened this Jun 12, 2024
@3142008956 3142008956 changed the base branch from master to executor_rewrite June 12, 2024 03:45
Copy link
Contributor

@BrandonWong14 BrandonWong14 left a comment

Choose a reason for hiding this comment

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

Looks good, we can deal with the failing tests to see if its still failing after the rest are updated.

@BrandonWong14 BrandonWong14 merged commit 9a558ab into executor_rewrite Jun 12, 2024
1 check failed
@BrandonWong14 BrandonWong14 deleted the student_code_update branch June 12, 2024 03:51
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.

2 participants