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

changed line terminator to be an array #452

Merged
merged 3 commits into from Apr 7, 2015
Merged

Conversation

JeniT
Copy link

@JeniT JeniT commented Apr 5, 2015

This is to support the ‘best practice CSV’ which allows either CRLF or
LF line terminators.

This is to support the ‘best practice CSV’ which allows either CRLF or
LF line terminators.
@gkellogg
Copy link
Member

gkellogg commented Apr 5, 2015

I'll need to update the namespace too.

…tors`. Still requires @iherman to update the property diagram.
@gkellogg
Copy link
Member

gkellogg commented Apr 6, 2015

Still requires that @iherman update the property diagram to be consistent.

@gkellogg gkellogg assigned iherman and unassigned gkellogg Apr 6, 2015
@iherman
Copy link
Member

iherman commented Apr 6, 2015

Indeed. I have already changed the 'source' file on my disk, I have also removed header column count from the dialect. I am still waiting on the outcome of issue #444 which may also lead to a change of the diagram before I generate the SVG and PNG files.

@iherman
Copy link
Member

iherman commented Apr 7, 2015

Diagram has been updated in commit 8f9caca

…inators

# Conflicts:
#	ns/_vocab.csv
#	tests/manifest.csv
gkellogg added a commit that referenced this pull request Apr 7, 2015
changed line terminator to be an array
@gkellogg gkellogg merged commit f99acae into gh-pages Apr 7, 2015
@gkellogg gkellogg deleted the jt-tech-line-terminators branch April 7, 2015 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants