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

Fixes to the doxygen documentation INPUT line #264

Merged
merged 1 commit into from
Mar 1, 2024
Merged

Fixes to the doxygen documentation INPUT line #264

merged 1 commit into from
Mar 1, 2024

Conversation

leonardocarreras
Copy link
Collaborator

With the new structure in PR #94, that is:

  • dpsim
  • dpsim-models
  • dpsim-villas each one has a src and include folders. This breaks the INPUT argument in Doxygen.

This commit fixes this and resolves #260.

@leonardocarreras leonardocarreras added bug Something isn't working documentation Improvements or additions to documentation labels Nov 10, 2023
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Dec 9, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

warning The version of Java (11.0.14.1) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

@stv0g stv0g self-requested a review February 26, 2024 12:56
stv0g
stv0g approved these changes Feb 26, 2024
View reviewed changes
Copy link
Contributor

@stv0g stv0g left a comment

Choose a reason for hiding this comment

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

LGTM

@stv0g
Copy link
Contributor

stv0g commented Feb 29, 2024

@m-mirz Do you mind merging this? Its a relatively small change.

@leonardocarreras @m-mirz
Fixes the doxygen documentation input line
2f92013 
With the new structure in PR #94, that is:
* dpsim
* dpsim-models
* dpsim-villas
each one has a src and include folders. This breaks the INPUT argument in Doxygen.

This commit fixes #260.

Signed-off-by: Leonardo Carreras <leonardo.carreras@eonerc.rwth-aachen.de>
@m-mirz m-mirz force-pushed the fix-doxygen-input-paths branch from c7557d4 to 2f92013 Compare March 1, 2024 16:41
@m-mirz m-mirz merged commit 29c0145 into master Mar 1, 2024
9 checks passed
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Mar 1, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@stv0g stv0g deleted the fix-doxygen-input-paths branch March 7, 2024 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stv0g stv0g stv0g approved these changes

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

Assignees

@dinkelbachjan dinkelbachjan

@leonardocarreras leonardocarreras

Labels
bug Something isn't working documentation Improvements or additions to documentation
Projects
DPsim
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Fix Doxygen documentation
4 participants
@leonardocarreras @stv0g @m-mirz @dinkelbachjan