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

HTML to Mardown #54

Closed
wants to merge 1 commit into from
Closed

Conversation

lvisintini
Copy link
Contributor

No description provided.

@guidokessels
Copy link
Owner

Hey, this is great. I'm a big fan of Markdown.

However, I'm not really sure if Markdown is a good fit for this project. It means that every project that uses xwing-data will have to preprocess the files before using them. Right now xwing-data can be dropped into most (web) projects as-is, which is great. You could even load the files directly from GitHub and it'll work.

So I'm a bit torn on this. Markdown is much nicer than having HTML in the text fields, but I think it makes xwing-data a bit harder to use.

@lvisintini
Copy link
Contributor Author

I hear you. you are right.
Do you like the idea enough to manage a parallel branch with a markdown flavour of the data?
I have never done anything like that and I'm sure it wont be straight forward, but I would interested in helping out on something like that just because I have never done anything of the sort.

@guidokessels
Copy link
Owner

I don't think having separate branches is good idea. It's tough to maintain and keep in sync.

I'm going to close this pr, but feel free to keep the discussion going.

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.

None yet

2 participants