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

Extract embed element #47

Merged
merged 1 commit into from
Sep 14, 2017
Merged

Extract embed element #47

merged 1 commit into from
Sep 14, 2017

Conversation

dsager
Copy link
Member

@dsager dsager commented Sep 14, 2017

  • Add Embed element, which can be re-used later on
  • Additional test cases for EmbeddedParser.build
  • Additional test cases for EmbeddedParser.supported?
  • Rename some internal methods in EmbeddedParser

- Add `Embed` element, which can be re-used later on
- Additional test cases for `EmbeddedParser.build`
- Additional test cases for `EmbeddedParser.supported?`
- Rename some internal methods in `EmbeddedParser`
@towanda
Copy link
Contributor

towanda commented Sep 14, 2017

+1

1 similar comment
@ifosch
Copy link
Member

ifosch commented Sep 14, 2017

+1

@dsager dsager merged commit 073ff3d into Devex:master Sep 14, 2017
@dsager dsager deleted the embedded-element branch September 14, 2017 10:07
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

3 participants