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

fail to add dependencies to ccov-all with ${CMAKE_PROJECT_NAME}_docs #38

Closed
msclock opened this issue Nov 1, 2023 · 0 comments · Fixed by #39
Closed

fail to add dependencies to ccov-all with ${CMAKE_PROJECT_NAME}_docs #38

msclock opened this issue Nov 1, 2023 · 0 comments · Fixed by #39

Comments

@msclock
Copy link
Collaborator

msclock commented Nov 1, 2023

Because the lack of doxygen, no ${CMAKE_PROJECT_NAME}_docs target is generated , so it failed to add the target as a pre-dependency to ccov-all of the coverage configuration.
msclock added a commit that referenced this issue Nov 1, 2023
@msclock
fix: add _docs as a pre-denpendency
d031319 
Close #38

Signed-off-by: msclock <msclock@qq.com>
@msclock msclock mentioned this issue Nov 1, 2023
Merged
msclock added a commit that referenced this issue Nov 1, 2023
@msclock
fix: add a pre-dependency for ccov-all
3d24f7b 
Fail to add ${CMAKE_PROJECT_NAME}_docs as a pre-dependency
because of the the possibility of the lack of doxygen and
no generated target of ${CMAKE_PROJECT_NAME}_docs to add.

Close #38

Signed-off-by: msclock <msclock@qq.com>
msclock added a commit that referenced this issue Nov 1, 2023
@msclock
fix: add a pre-dependency for ccov-all
d960035 
Fail to add ${CMAKE_PROJECT_NAME}_docs as a pre-dependency
because of the the possibility of the lack of doxygen and
no generated target of ${CMAKE_PROJECT_NAME}_docs to add.

Close #38

Signed-off-by: msclock <msclock@qq.com>
github-actions bot pushed a commit that referenced this issue Nov 1, 2023
@semantic-release-bot
chore(release): 1.2.7 [skip ci]
86fe399 
## [1.2.7](v1.2.6...v1.2.7) (2023-11-01)

### Bug Fixes

* add a pre-dependency for ccov-all ([d960035](d960035)), closes [#38](#38)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: add a pre-dependency for ccov-all serious-scaffold/ss-cpp
1 participant
@msclock