Skip to content

tests: Check that example symbol isn't released#2135

Merged
openshift-merge-robot merged 1 commit intoostreedev:masterfrom
mwleeds:test-symbols-check-for-example
Jun 18, 2020
Merged

tests: Check that example symbol isn't released#2135
openshift-merge-robot merged 1 commit intoostreedev:masterfrom
mwleeds:test-symbols-check-for-example

Conversation

@mwleeds
Copy link
Copy Markdown
Member

@mwleeds mwleeds commented Jun 17, 2020

For the motivation for this see #2132.

@mwleeds mwleeds force-pushed the test-symbols-check-for-example branch 2 times, most recently from cec1f29 to 98d5353 Compare June 17, 2020 22:13
@mwleeds mwleeds force-pushed the test-symbols-check-for-example branch from 98d5353 to d211816 Compare June 17, 2020 22:28
@mwleeds
Copy link
Copy Markdown
Member Author

mwleeds commented Jun 17, 2020

Had to fix some silly mistakes. Sorry, I'll test locally next time. I did check locally that the new library function assert_file_has_content_once works as advertised.

@cgwalters
Copy link
Copy Markdown
Member

Thank you!
/approve
/lgtm

@openshift-ci-robot
Copy link
Copy Markdown
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cgwalters, mwleeds

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-robot openshift-merge-robot merged commit 2aa5ec7 into ostreedev:master Jun 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants