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.

Sign up for GitHub

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

Jump to bottom

boards/nucleo-g07xrb: fix or exclude some doxygen warnings [backport 2021.10] #17084

Merged
merged 2 commits into from Oct 29, 2021
Merged

boards/nucleo-g07xrb: fix or exclude some doxygen warnings [backport 2021.10] #17084

merged 2 commits into from Oct 29, 2021

Conversation

kaspar030
Copy link
Contributor

Backport of #17079

Contribution description

This PR fixes some doxygen warnings, and adds the remaining patterns that focal's doxygen still complains.
For future use, a commit also adds a script to create the patterns from doxygen warnings, with the logic taken from #16779 (comment). (split out into #17082)

Testing procedure

CI should be fine.

Issues/PRs references

RIOT-OS/riotdocker#104

@kaspar030 kaspar030 added Area: boards Area: Board ports Area: tools Area: Supplementary tools CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Process: release backport Integration Process: The PR is a release backport of a change previously provided to master Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) labels Oct 29, 2021
fjmolinas
fjmolinas approved these changes Oct 29, 2021
View reviewed changes
Copy link
Contributor

@fjmolinas fjmolinas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK!

@kaspar030 kaspar030 merged commit fad46eb into RIOT-OS:2021.10-branch Oct 29, 2021
@kaspar030 kaspar030 deleted the backport/2021.10/exclude_some_dox_warnings branch October 29, 2021 18:07
@fjmolinas fjmolinas added this to the Release 2022.01 milestone Nov 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fjmolinas fjmolinas fjmolinas approved these changes

@aabadie aabadie Awaiting requested review from aabadie aabadie is a code owner

Assignees
No one assigned
Labels
Area: boards Area: Board ports Area: tools Area: Supplementary tools CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Process: release backport Integration Process: The PR is a release backport of a change previously provided to master Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Milestone
Release 2022.01
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@kaspar030 @fjmolinas