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

Add macro yaml_str #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add macro yaml_str #27

wants to merge 1 commit into from

Conversation

tshort
Copy link

@tshort tshort commented Nov 27, 2015

It could use a test, but I wasn't sure how to integrate it.

I'm also trying to add support in the Atom editor (JuliaEditorSupport/atom-language-julia#52). Assuming no major objections, I'll try for Textmate/Sublime support next.

@tshort
Copy link
Author

tshort commented Dec 15, 2015

Pinging...

@sglyon
Copy link
Collaborator

sglyon commented Feb 23, 2017

Hey @tshort thanks for working on this.

Sorry it has sat for so long. What was the status with the editor plugins and JSON.jl. did a corresponding change get merged anywhere else?

@tshort
Copy link
Author

tshort commented Feb 23, 2017

Thanks. There's been no action in the other packages in response to PRs. Maybe the approach is trying to be too clever.

@kescobo
Copy link
Collaborator

kescobo commented Aug 27, 2020

@tshort Trying to get through PR backlog here - I think having a yaml string macro would be quite nice. Is this still needed / desired? Any chance you'd be willing to update for the things that have changed in the last 3+ years? 🙏

@tshort
Copy link
Author

tshort commented Aug 27, 2020

I don't have an immediate need, but I can take a look and try to add a test.

@kescobo
Copy link
Collaborator

kescobo commented Aug 27, 2020

That would be great, thank you!

@kescobo kescobo removed the question label Aug 27, 2020
@kescobo kescobo self-requested a review August 27, 2020 16:44
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.

3 participants