eclipse-score / eclipse-score/lifecycle

Fix missing includes & bazel target dependencies

Open
#301 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
6
Forks
34
Avg merge
2d 3h
Merged PRs (30d)
80

Description

### What

We have a lot of missing/not needed includes which should be fixed.
There are also some Bazel targets that rely on dependencies inherited from other dependencies which should also not happen.

Would be great if we had CI to check and reject changes that break this.

### Acceptance Criteria (DoD)

- Includes fixed.
- All Bazel targets shall depend on what they use.
- CI setup to reject bad changes.

### How

- [include-what-you-use](https://github.com/include-what-you-use/include-what-you-use) is a standard tool to fix the includes.
- [depend_on_what_you_use](https://github.com/martis42/depend_on_what_you_use) Is a similar tool I've found but not sure how know this is.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.