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

Address more warnings #605

Merged
merged 25 commits into from
Jul 1, 2024
Merged

Conversation

wxmerkt
Copy link
Contributor

@wxmerkt wxmerkt commented Jun 29, 2024

Seems like quite a few slipped through in #601. May need another iteration or so after this one.

nim65s
nim65s previously approved these changes Jun 30, 2024
@jcarpent jcarpent merged commit a6ae155 into coal-library:devel Jul 1, 2024
29 of 33 checks passed
@wxmerkt
Copy link
Contributor Author

wxmerkt commented Jul 6, 2024

@jcarpent @nim65s Can we get a patch release please with the latest PRs merged before the library gets renamed? Thank you :-)

@nim65s
Copy link
Contributor

nim65s commented Jul 6, 2024

it can't be a patch release, there are breaking changes in devel already. But we can make hpp-fcl v3.0.0 along with coal. Would it work for you ?

@wxmerkt
Copy link
Contributor Author

wxmerkt commented Jul 6, 2024

Ah, I didn't notice the breaking changes. Can we have another release of hpp-fcl before the renaming or will this have to go together based on the changes?

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.

3 participants