Skip to content

DRY reading yaml - #64

Merged
eshellman merged 5 commits into
book-postfrom
simplify
Mar 25, 2018
Merged

DRY reading yaml#64
eshellman merged 5 commits into
book-postfrom
simplify

Conversation

@eshellman

Copy link
Copy Markdown
Contributor

don't have classes reading yaml in different ways.

don't have classes reading yaml in different ways.
@bdr99

bdr99 commented Mar 19, 2018

Copy link
Copy Markdown
Contributor

Thanks, I wasn't aware of the pandata library.

@eshellman
eshellman merged commit e9e4eda into book-post Mar 25, 2018
@bdr99

bdr99 commented Mar 25, 2018

Copy link
Copy Markdown
Contributor

I am taking the changes from this PR and moving them into #63 because I need them in order to do some of the restructuring that I am trying to do there, so I am going to close this PR.

@eshellman

Copy link
Copy Markdown
Contributor Author

whoops I merged this - hope that doesn't mess you up

@bdr99

bdr99 commented Mar 25, 2018

Copy link
Copy Markdown
Contributor

No, it's fine either way. I just noticed that this PR is into the book-post branch, not master. Was there a reason you did that?

@eshellman

Copy link
Copy Markdown
Contributor Author

because I didn't want to create merge conflicts for book-post

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.

2 participants