Schema corrections by @madskristensen without tabs/spaces

[ocram]

Fixes the same issues that @madskristensen fixed in #105

But without:

• indentation changes (tabs/spaces)
• changes to email section which have become obsolete
• changes to profiles section which did neither match the current use in resume.json nor the style from email

[madskristensen]

Please keep in mind that by not changing email and profile, the thomasdavis.json example on jsonresume.org is not valid.

[ocram]

The example on jsonresume.org is just outdated. The homepage is not updated as often as this repository. The only file that must be valid in any case is resume.json, because it is validated against the schema in the tests.

[DonDebonair]

+1 for merging this!

[thomasdavis]

Okay, I will merge this soon, have to do write up the Changelog for it. We didn't really get a consensus vote for awardDate -> date but it makes sense and we will just go with it.

[thomasdavis]

@madskristensen Thanks again, we are doing a quick release of a 0.0.0 version, once that is in place, everything will be updated. Then all modules will follow the versioning system and update accordingly.

[thomasdavis]

I will also do a write up once this is done, and clone the schemastore.org to make sure everything is in sync.

[madskristensen]

Sounds good. Let me know if I can help

[thomasdavis]

Schema is looking much healthier now. The last question is, I wonder if we
should enforce additionalProperties: false. I don't really have any qualms
with developers extending their resumes as long as they understand the
consequences.

On Tue, Jul 15, 2014 at 2:39 PM, Mads Kristensen notifications@github.com
wrote:

Sounds good. Let me know if I can help

—
Reply to this email directly or view it on GitHub
#107 (comment)
.

Thomas Davis
http://thomasdav.is

VP of Tech - Earbits - http://earbits.com
Co-founder - Cdnjs - http://cdnjs.com
Founder - Backbone Tutorials - http://backbonetutorials.com

[thomasdavis]

Also added an issue to document that are now available on SchemaStore.org

#108

On Tue, Jul 15, 2014 at 2:41 PM, Thomas Davis thomasalwyndavis@gmail.com
wrote:

Schema is looking much healthier now. The last question is, I wonder if we
should enforce additionalProperties: false. I don't really have any qualms
with developers extending their resumes as long as they understand the
consequences.

On Tue, Jul 15, 2014 at 2:39 PM, Mads Kristensen <notifications@github.com

wrote:

Sounds good. Let me know if I can help

—
Reply to this email directly or view it on GitHub
#107 (comment)
.

Thomas Davis
http://thomasdav.is

VP of Tech - Earbits - http://earbits.com
Co-founder - Cdnjs - http://cdnjs.com
Founder - Backbone Tutorials - http://backbonetutorials.com

Thomas Davis
http://thomasdav.is

VP of Tech - Earbits - http://earbits.com
Co-founder - Cdnjs - http://cdnjs.com
Founder - Backbone Tutorials - http://backbonetutorials.com

[madskristensen]

If additionalProperties are not set to false, then validation will be tough, because almost any JSON file will validate against the schema

[ocram]

... and the tools/themes will generally not make use of any additional properties. Apart from that, the more permissive the specification is, the less powerful and successful will the standard be (due to fragmentation, confusion, etc.), right?

[thomasdavis]

Yep, the tests should be strict for tools that are trying to adhere to the
standard. If people are willing to deviate they will have to deal with
broken test for now.

On Tue, Jul 15, 2014 at 2:46 PM, Marco notifications@github.com wrote:

... and the tools/themes will generally not make use of any additional
properties. Apart from that, the more permissive the specification is, the
less powerful and successful will the standard be (due to fragmentation,
confusion, etc.), right?

—
Reply to this email directly or view it on GitHub
#107 (comment)
.

Thomas Davis
http://thomasdav.is

VP of Tech - Earbits - http://earbits.com
Co-founder - Cdnjs - http://cdnjs.com
Founder - Backbone Tutorials - http://backbonetutorials.com

[madskristensen]

You probably also want to set "required": ["bio"] so empty documents won't validate either. That's the case now as you can see here http://schemastore.org/validator/#/schemas/json/resume

[thomasdavis]

Good idea, and also, awesome link! We were using our CLI tool for test but
the GUI makes it pretty easy to set errors.

On Tue, Jul 15, 2014 at 2:54 PM, Mads Kristensen notifications@github.com
wrote:

You probably also want to set "required": ["bio"] so empty documents
won't validate either. That's the case now as you can see here
http://schemastore.org/validator/#/schemas/json/resume

—
Reply to this email directly or view it on GitHub
#107 (comment)
.

Thomas Davis
http://thomasdav.is

VP of Tech - Earbits - http://earbits.com
Co-founder - Cdnjs - http://cdnjs.com
Founder - Backbone Tutorials - http://backbonetutorials.com

[madskristensen]

I'm glad you like it. I've just published it a few minutes ago. Still a work in progress. Let me know if you have any ideas for improvements