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 doxygen to 1.9.6 #1844

Merged
merged 10 commits into from
Jul 5, 2023
Merged

Update doxygen to 1.9.6 #1844

merged 10 commits into from
Jul 5, 2023

Conversation

kstribrnAmzn
Copy link
Member

@kstribrnAmzn kstribrnAmzn commented Dec 22, 2022

Description of changes:
Update doxygen to 1.9.6 using doxygen -u command

Testing
Not tested yet.

By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.

@kstribrnAmzn kstribrnAmzn requested a review from a team as a code owner December 22, 2022 20:21
@kstribrnAmzn
Copy link
Member Author

Doxygen check failures are expected until submodules are updated. May pull latest submodules as part of this change.

@kstribrnAmzn kstribrnAmzn changed the title Update doxygen to 1.9.4 Update doxygen to 1.9.6 Jan 4, 2023
The previous coreMQTT submodule had
a higher version as the submodule
was the LTS release.
@kstribrnAmzn
Copy link
Member Author

Manifest verifier will break until the all of the submodules have a new tag which aliases the commit pinning doxygen to 1.9.6.

Will need to bring this up with the team tomorrow. Tagging should take minutes to do but I'm not sure of further implications.

@kstribrnAmzn
Copy link
Member Author

Tags are only added when a release is performed. Leaving this open so that it gets pulled in during the next release.

@kstribrnAmzn
Copy link
Member Author

Got agreement to use commit hashes instead of tags on main. Submitting for approval.

@kstribrnAmzn kstribrnAmzn merged commit 52f2d40 into aws:main Jul 5, 2023
@kstribrnAmzn kstribrnAmzn deleted the doxygen-update branch July 5, 2023 23:38
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