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

compat fixes #93

Merged
merged 3 commits into from
Oct 31, 2020
Merged

compat fixes #93

merged 3 commits into from
Oct 31, 2020

Conversation

mgkuhn
Copy link
Contributor

@mgkuhn mgkuhn commented Oct 18, 2020

not really useful here since this package has hardly any dependencies
a change of the compat section does not count as just a patch,
so we need to increase the minor version of this package
to fix it
@coveralls
Copy link

Coverage Status

Coverage remained the same at 68.421% when pulling ae1e49c on mgkuhn:compat into 9322315 on dancasimiro:main.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 68.421% when pulling ae1e49c on mgkuhn:compat into 9322315 on dancasimiro:main.

@dancasimiro
Copy link
Owner

Thanks @mgkuhn. I must admit that I don't follow the need to restrict the package to Julia 1.4 & 1.5. The unit tests appeared to pass on the older versions, but I have admittedly not smoke tested the current package on the historical releases. I'm still wrapping my head around the new compat specifications and mistakenly excluded the intermediate 1.x releases. I had intended to include all versions from 0.7 to the current 1.5.

@dancasimiro dancasimiro merged commit 249b072 into dancasimiro:main Oct 31, 2020
@terasakisatoshi
Copy link

Thank you it worked.

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.

wavplay(y, fs) does not work
4 participants