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

Performing changes for v0.3.2 #95

Merged
merged 9 commits into from
Sep 18, 2020
Merged

Performing changes for v0.3.2 #95

merged 9 commits into from
Sep 18, 2020

Conversation

GeorgFerdinandSchneider
Copy link
Collaborator

Update of bot towards v0.3.2

-> revision of labels -> closes #90 #88 #87 #64 #47
-> removed FOAF -> closed #11
-> version iris changes -> closes #80

added labels and comments as of 31/07/2020 pending #90 #88 #87 #47
Adding intermediary step of multi language labels
remove foaf:Person and foaf:name

substituted with vcard and schema.org annotation

removed bnode fashion of stating contributors and creators
@madnificent
Copy link

My eye did not fall on any immediate issues content-wise. 🎉

Not sure if there are indentation guidelines for the ttl files in this project but I noticed there are inconsistencies in the indentation of the prefixes in these diffs.

In order to lower merge conflicts it probably makes sense to ignore these and/or to use a single space after a :. A different choice seems to have been made in the past, so you may want to stay consistent.

@GeorgFerdinandSchneider
Copy link
Collaborator Author

@madnificent Thanks Aad for the comment! Good point with the indentation. I am not aware if there is any guideline for ttl. -> If merged to master the CI of Maxime recreates all files so may be the different indentation is not so much of an issue?

Copy link
Member

@maximelefrancois86 maximelefrancois86 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok!

@GeorgFerdinandSchneider GeorgFerdinandSchneider changed the title Develop Performing changes for v0.3.2 Sep 18, 2020
@GeorgFerdinandSchneider GeorgFerdinandSchneider merged commit a460732 into master Sep 18, 2020
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.

BOT Version IRIs do not resolve Update definition of bot:hasSpace foaf:name
3 participants