Skip to content

Add property with "log info"#148

Merged
bastibe merged 1 commit intomasterfrom
info-property
Oct 16, 2015
Merged

Add property with "log info"#148
bastibe merged 1 commit intomasterfrom
info-property

Conversation

@mgeier
Copy link
Contributor

@mgeier mgeier commented Oct 2, 2015

See #23 and #58.

@mgeier mgeier added this to the 0.8.0 milestone Oct 2, 2015
@mgeier
Copy link
Contributor Author

mgeier commented Oct 4, 2015

I replaced 5a8da5e with 3a54a75.

Now the method is called extra_info() (in order to not be confused with soundfile.info() from #58) and I added a small test.

Is it OK to merge?

@mgeier
Copy link
Contributor Author

mgeier commented Oct 4, 2015

I forgot to rename the property in the commit message, too.
I replaced 3a54a75 with cef2fdd, I hope now it's really OK.

bastibe added a commit that referenced this pull request Oct 16, 2015
@bastibe
Merge pull request #148 from bastibe/info-property
44ebbd5 
Add property with "log info"
@bastibe bastibe merged commit 44ebbd5 into master Oct 16, 2015
@mgeier mgeier deleted the info-property branch October 17, 2015 06:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

0.8.0

Development

Successfully merging this pull request may close these issues.

2 participants

@mgeier @bastibe

Comments