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.

Sign up for GitHub

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

Jump to bottom

Moved performance tests to dedicated folder #121

Merged
merged 2 commits into from
Feb 17, 2017
Merged

Moved performance tests to dedicated folder #121

merged 2 commits into from
Feb 17, 2017

Conversation

favreau
Copy link
Member

@favreau favreau commented Feb 16, 2017

No description provided.

@hernando
Copy link
Contributor

If the test is in a directory called perf you don't need to prefix its name by perf.

@favreau
Copy link
Member Author

favreau commented Feb 16, 2017

Yes, I remember that, but that still makes sense to start the name of the file with perf.

@favreau favreau force-pushed the latest branch 2 times, most recently from 5f60eb8 to d72c589 Compare February 16, 2017 14:40
@hernando
Copy link
Contributor

For me it's redundant.

@favreau
Copy link
Member Author

favreau commented Feb 17, 2017

Renamed perfBrayns.cpp to renderingSettings.cpp

@hernando hernando merged commit c77b039 into BlueBrain:master Feb 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hernando hernando hernando approved these changes

Assignees

@hernando hernando

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@favreau @hernando