Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Throw on unsupported format. #831

Merged
merged 4 commits into from
Dec 6, 2017
Merged

Throw on unsupported format. #831

merged 4 commits into from
Dec 6, 2017

Conversation

tech4him1
Copy link
Contributor

@tech4him1 tech4him1 commented Nov 18, 2017

- Summary

Closes #797.

If a format is specified that is unsupported, throw instead of just saving as markdown/frontmatter.

(Re)based on #795.

- Test plan

No regressions seen on /example site.
Threw if unsupported format set.

- Description for the changelog

Throw on unsupported format.

- A picture of a cute animal (not mandatory but encouraged)

@tech4him1 tech4him1 changed the title WIP: Throw on unsupported format. Throw on unsupported format. Nov 18, 2017
Copy link
Contributor

@erquhart erquhart left a comment

Choose a reason for hiding this comment

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

LGTM - temporarily setting to "request changes" to block until 0.8.0 is released with #795 merged.

@tech4him1 tech4him1 force-pushed the throw-unsupported-format branch from 07e9b5d to 2ecc535 Compare December 1, 2017 16:07
@tech4him1 tech4him1 changed the title Throw on unsupported format. WIP: Throw on unsupported format. Dec 1, 2017
@tech4him1
Copy link
Contributor Author

Looks like this is broken -- please don't merge.

@erquhart
Copy link
Contributor

erquhart commented Dec 1, 2017

We won't merge as long as it's WIP, no worries.

@tech4him1 tech4him1 force-pushed the throw-unsupported-format branch from 9301ca5 to d21ae0e Compare December 1, 2017 23:59
@tech4him1 tech4him1 changed the title WIP: Throw on unsupported format. Throw on unsupported format. Dec 2, 2017
@tech4him1
Copy link
Contributor Author

OK, this should be ready for review again. The markdown and html explicit formats have been removed -- format: frontmatter and extension: md are the defaults, and you can set extension: html if you prefer.

@tech4him1
Copy link
Contributor Author

@erquhart Please re-review before merge -- a few new commits were added.

@erquhart erquhart force-pushed the throw-unsupported-format branch from dc59840 to dede8e0 Compare December 6, 2017 15:08
@verythorough
Copy link
Contributor

Deploy preview ready!

Built with commit dede8e0

https://deploy-preview-831--netlify-cms-www.netlify.com

@erquhart erquhart merged commit 834c5f4 into master Dec 6, 2017
@erquhart erquhart deleted the throw-unsupported-format branch December 6, 2017 15:23
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Break on unsupported file format.
3 participants