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

Update README #69

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update README #69

wants to merge 1 commit into from

Conversation

MonJamp
Copy link
Collaborator

@MonJamp MonJamp commented May 26, 2017

@JethroDevon I've changed the message of the README a bit, since it's still your project I'm curious to know what you think.

@MonJamp MonJamp requested a review from JethroDevon May 26, 2017 04:34
@Dyrand
Copy link
Collaborator

Dyrand commented May 26, 2017 •
edited
Loading

We have to change some things if we really want to say we're using C++11, imo.
One example is for randomly selecting a song. We currently use rand, but we could use a random engine and some of the other cool things that the standard random library has.

@MonJamp
Copy link
Collaborator Author

MonJamp commented May 26, 2017

@Dyrand by the time @JethroDevon reviews this PR we could include more C++11 features.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JethroDevon JethroDevon Awaiting requested review from JethroDevon

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MonJamp @Dyrand