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

feat: thefederalistpapers parser #101

Merged
merged 6 commits into from
Feb 7, 2017
Merged

Conversation

janetleekim
Copy link
Contributor

Looks good - there is a “Read More” section that I would have liked to
clean out, but it’s embedded within the main body so I couldn’t get rid
of it.

Looks good - there is a “Read More” section that I would have liked to
clean out, but it’s embedded within the main body so I couldn’t get rid
of it.
@kev5873 kev5873 self-assigned this Feb 3, 2017
Copy link
Contributor

@dviramontes dviramontes left a comment

Choose a reason for hiding this comment

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

Test and preview look good, just needs the conflict resolved.

Copy link
Contributor

@dviramontes dviramontes left a comment

Choose a reason for hiding this comment

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

👍

@kev5873 kev5873 merged commit 7650324 into master Feb 7, 2017
@kev5873 kev5873 deleted the feat-thefederalistpapers-extractor branch February 7, 2017 19:30
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.

5 participants