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

Add Aqua and fix arising issues #67

Merged
merged 5 commits into from
Sep 27, 2023
Merged

Add Aqua and fix arising issues #67

merged 5 commits into from
Sep 27, 2023

Conversation

jonschumacher
Copy link
Member

No description provided.

@jonschumacher
Copy link
Member Author

@tknopp To make this work I had to remove the re-export for ImageMetadata since it exports data which results in an undefined export. Is this fine for you?

@tknopp
Copy link
Member

tknopp commented Sep 26, 2023

yes, I removed reexport altogether since it hides dependencies. Important is to do a correct version bump before releasing since it will break depending packages. I have already bumped the version on master.

@jonschumacher jonschumacher merged commit 3bd3f39 into master Sep 27, 2023
5 of 6 checks passed
@jonschumacher jonschumacher deleted the JS/add-aqua branch September 27, 2023 06:44
@tknopp
Copy link
Member

tknopp commented Sep 27, 2023

I am not sure, why the documentation building gives a failure. Maybe that is aqua related.

@jonschumacher
Copy link
Member Author

It is due to a major release of Documenter: https://github.com/JuliaDocs/Documenter.jl/blob/v1.0.0/CHANGELOG.md#breaking
I will fix it prior to the release.

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.

2 participants