Skip to content

Conversation

@boegel
Copy link
Member

@boegel boegel commented Dec 21, 2022

No description provided.

Comment on lines +4417 to +4418
print("Skipping test_det_copy_ec_specs using --from-pr, no GitHub token available?")
return
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
print("Skipping test_det_copy_ec_specs using --from-pr, no GitHub token available?")
return
self.skipTest("Not using --from-pr, no GitHub token available?")

See my longstanding work at #3790

Copy link
Member Author

Choose a reason for hiding this comment

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

I would rather change that across the board at once, so please update #3790 accordingly once this is merged.

I'm sorry for not getting to #3790 yet, we have to pick our battles (and this PR really should go in ASAP)

Copy link
Contributor

Choose a reason for hiding this comment

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

Agreed

@easybuilders easybuilders deleted a comment from boegelbot Dec 21, 2022
Copy link
Contributor

@akesandgren akesandgren left a comment

Choose a reason for hiding this comment

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

LGTM

@akesandgren
Copy link
Contributor

Going in, thanks @boegel!

@akesandgren akesandgren merged commit 662c696 into easybuilders:develop Dec 21, 2022
@boegel boegel deleted the test_det_copy_ec_specs_no_token branch December 21, 2022 14:52
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.

3 participants