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

Add pytest.ini so local tests don't display warning #446

Merged
merged 1 commit into from
Jun 19, 2020

Conversation

clalancette
Copy link
Contributor

Signed-off-by: Chris Lalancette [email protected]

See ros2/ros2#951 for more details and CI.

Copy link
Collaborator

@emersonknapp emersonknapp left a comment

Choose a reason for hiding this comment

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

lgtm with CI

@clalancette
Copy link
Contributor Author

The failure in the Rolling PR job is a crash in rosbag2_transport.test_record__rmw_fastrtps_cpp . Totally unrelated to this change, so I'm going to go ahead and merge this. Thanks for the review.

@clalancette clalancette merged commit a4d49db into master Jun 19, 2020
@clalancette clalancette deleted the clalancette/add-pytest-ini branch June 19, 2020 12:21
emersonknapp pushed a commit that referenced this pull request Aug 24, 2020
emersonknapp pushed a commit that referenced this pull request Aug 24, 2020
emersonknapp pushed a commit that referenced this pull request Aug 24, 2020
emersonknapp added a commit that referenced this pull request Aug 26, 2020
Signed-off-by: Chris Lalancette <[email protected]>

Co-authored-by: Chris Lalancette <[email protected]>
emersonknapp added a commit that referenced this pull request Aug 26, 2020
Signed-off-by: Chris Lalancette <[email protected]>

Co-authored-by: Chris Lalancette <[email protected]>
emersonknapp added a commit that referenced this pull request Aug 26, 2020
Signed-off-by: Chris Lalancette <[email protected]>

Co-authored-by: Chris Lalancette <[email protected]>
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