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

UUID-class for UUIDs #24

Open

Dauerdaddlah opened this issue Jan 14, 2019 · 1 comment

Labels

good first issue

Member

Dauerdaddlah commented Jan 14, 2019 •

edited by Valefant

Loading

We should check if it may a good idea to use the UUID-class for UUIDs instead of simple Strings.

Using a dedicated class could clearly show what we want to have, instead of a 'primitive' type, that could be of any meaning.

If not using the class a typedef for UUID (internally still using a String) could possibly help make the code more readable

Dauerdaddlah added the evaluate label

Member

Valefant commented Jan 16, 2019

A typedef sounds reasonable 👍

Valefant added good first issue and removed evaluate labels

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