Skip to content
This repository was archived by the owner on Feb 17, 2023. It is now read-only.

Conversation

@rhattersley
Copy link
Member

No description provided.

This was referenced Jan 24, 2014
@bjlittle
Copy link
Member

Yup, solid 👍

bjlittle added a commit that referenced this pull request Jan 24, 2014
@bjlittle bjlittle merged commit 3fc1f0f into SciTools:master Jan 24, 2014
Copy link
Member

Choose a reason for hiding this comment

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

I see there are 3 (count'em!) underscores here.
It isn't really a full init test, but only tests the '.concrete' aspect, so arguably it should be called "Test___init____concrete".
( as in "Test_/name_of_public_method/__/aspect_of_method" --> "Test_/__init__/__/concrete". )

In fact, a good job this has been renamed, or it would be "Test___init_____concrete".
Five in a row -- could this be a record ?? ;-)

@rhattersley rhattersley deleted the expose-concrete branch January 24, 2014 15:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants