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

Website: warn against side effects in bare code #168

Open
BrainCrumbz opened this issue Feb 21, 2017 · 0 comments
Open

Website: warn against side effects in bare code #168

BrainCrumbz opened this issue Feb 21, 2017 · 0 comments

Comments

@BrainCrumbz
Copy link
Collaborator

BrainCrumbz commented Feb 21, 2017

See example gist.

When running from console, this line runs only once, during context building phase. And not on each example execution.

When running from VS IDE, that line runs everytime test discovery is requested, then everytime test execution is requested. And not on each example execution.

Such a warning should be placed in general in doc page, e.g. in some general conventions, or common rules, or similar. And then also in VS integration section, once PR #163 lands and that section will be needed as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant