Skip to content

Namespace cleanup. Fixes #37#61

Merged
MehdiK merged 3 commits intoTestStack:masterfrom
mwhelan:master
Mar 20, 2014
Merged

Namespace cleanup. Fixes #37#61
MehdiK merged 3 commits intoTestStack:masterfrom
mwhelan:master

Conversation

@mwhelan
Copy link
Copy Markdown
Member

@mwhelan mwhelan commented Mar 14, 2014

Rationalized namespaces down to 3.
TestStack.BDDfy
TestStack.BDDfy.Configuration
TestStack.BDDfy.Processors

@MehdiK
Copy link
Copy Markdown
Member

MehdiK commented Mar 17, 2014

Great work @mwhelan. This is much needed and very useful.

@mwhelan
Copy link
Copy Markdown
Member Author

mwhelan commented Mar 17, 2014

I think that having the main namespace lean is good as this will be the most frequently used in tests. Configuration and Processors are more to do with the initial setup, so they can be a bit less discoverable than Core and Scanners, which are used in tests. If they are less discoverable then that increases the need for good documentation explaining their use though.

@MehdiK MehdiK mentioned this pull request Mar 17, 2014
@MehdiK
Copy link
Copy Markdown
Member

MehdiK commented Mar 17, 2014

I am proposing an alternative solution in #62.

Basically what I've done, except the Scanners, I've restructured the project to look the same as the desired namespaces: moved the Core files out to the root and moved Reporters out of the Processors folder to the root and let the folder dictate the namespace on Reporters. Check it out and let me know what you think please.

@MehdiK MehdiK merged commit 79dfc37 into TestStack:master Mar 20, 2014
@mwhelan mwhelan mentioned this pull request Apr 9, 2014
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