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

Update dependencies according to breathe==4.35.0 #200

Closed
wants to merge 1 commit into from

Conversation

Rotzbua
Copy link
Contributor

@Rotzbua Rotzbua commented Sep 12, 2023

Change

  • Breaking: Remove support of sphinx 3.x
  • Update dependencies according to breathe==4.35.0

Fixes #202

@Chrismarsh
Copy link

Just noting that this is blocking C++ code doc generation on RTD for me, and having PR#205 merged would be fantastic.

@svenevs
Copy link
Owner

svenevs commented Jan 16, 2024

Thanks @Chrismarsh , fast response from a phone while out. I set a reminder to give this project some attention this weekend, please poke me by Sunday to get a new release out if I haven't gotten it done this week (a lot of stuff going on already...)

@Chrismarsh
Copy link

Thanks @svenevs . The work on the project is much apprecaited!
For anyone else running into this problem like #203 and #202 is to use this PR directly in your requirements
exhale @ git+https://github.com/svenevs/exhale.git@refs/pull/205/merge

@svenevs svenevs closed this in #205 Jan 21, 2024
@svenevs
Copy link
Owner

svenevs commented Jan 21, 2024

@Chrismarsh I released 0.3.7.... I broke CI (#212), please let me know if the official release actually works for you 🙏

@Chrismarsh
Copy link

Ah great! I'm afk this week and will try it next week and let you know!

@svenevs
Copy link
Owner

svenevs commented Jan 23, 2024

No problem, thanks for the response! I think if I had broken everything somebody probably would have raised an issue by now 🙃 But you'll likely want to set your requirements to exhale>=0.3.7 at some point so feedback that it works would be much appreciated.

Chrismarsh added a commit to Chrismarsh/CHM that referenced this pull request Jan 29, 2024
@Chrismarsh
Copy link

@svenevs just made the change and built on RTD. Confirmed exhale==0.3.7 was pulled. All seems to "just work" as expected. Thanks!

@svenevs
Copy link
Owner

svenevs commented Jan 30, 2024

Hooray! Thanks for following up and letting me know 🙃

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.

Problem with Sphinx 7
3 participants