Skip to content

[rules score] make example separate modules#350

Open
hoe-jo wants to merge 3 commits into
mainfrom
joho_examples_separate_modules
Open

[rules score] make example separate modules#350
hoe-jo wants to merge 3 commits into
mainfrom
joho_examples_separate_modules

Conversation

@hoe-jo

@hoe-jo hoe-jo commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@hoe-jo
hoe-jo force-pushed the joho_examples_separate_modules branch 2 times, most recently from 3ba0969 to 83d6358 Compare July 15, 2026 12:29
@github-actions

Copy link
Copy Markdown

Coverage Report

Coverage report was generated.

Full report can be downloaded from the CI artifacts (expand Artifacts at the bottom of the run).

Overall coverage rate:

lines......: 83.9%
functions......: 52.0%

@github-actions

Copy link
Copy Markdown

Coverage Report

Coverage report was generated.

Full report can be downloaded from the CI artifacts (expand Artifacts at the bottom of the run).

Overall coverage rate:

lines......: 83.9%
functions......: 52.0%

@hoe-jo
hoe-jo force-pushed the joho_examples_separate_modules branch from 594be84 to d68d981 Compare July 15, 2026 14:08
@github-actions

Copy link
Copy Markdown

Coverage Report

Coverage report was generated.

Full report can be downloaded from the CI artifacts (expand Artifacts at the bottom of the run).

Overall coverage rate:

lines......: 83.9%
functions......: 52.0%

Comment thread .github/workflows/tests.yml Outdated
Comment on lines +99 to +106
- name: Run dependable element examples
run: |
cd bazel/rules/rules_score/examples/seooc
bazel build //...
bazel test //...
cd ../some_other_library
bazel build //...
bazel test //...

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Highly recommend to make this separate jobs. We don't have issues with parallel runners and this will reduce runtime and will limit cache destruction.

Comment thread bazel/rules/rules_score/examples/some_other_library/MODULE.bazel
Comment thread MODULE.bazel Outdated
remote = "https://github.com/bmw-software-engineering/lobster.git",
)

bazel_dep(name = "rules_distroless", version = "0.8.0") # For

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Comment artifact?

- create separate bazel modules
- include them in docu
@hoe-jo
hoe-jo force-pushed the joho_examples_separate_modules branch from d68d981 to 5e4f550 Compare July 20, 2026 11:37
- no bazel solution for libcairo available
@hoe-jo
hoe-jo force-pushed the joho_examples_separate_modules branch from 5e4f550 to 150433b Compare July 20, 2026 11:42
@github-actions

Copy link
Copy Markdown

Coverage Report

Coverage report was generated.

Full report can be downloaded from the CI artifacts (expand Artifacts at the bottom of the run).

Overall coverage rate:

lines......: 83.9%
functions......: 51.9%

@github-actions

Copy link
Copy Markdown

Coverage Report

Coverage report was generated.

Full report can be downloaded from the CI artifacts (expand Artifacts at the bottom of the run).

Overall coverage rate:

lines......: 83.9%
functions......: 51.9%

@github-actions

Copy link
Copy Markdown

Coverage Report

Coverage report was generated.

Full report can be downloaded from the CI artifacts (expand Artifacts at the bottom of the run).

Overall coverage rate:

lines......: 83.9%
functions......: 51.9%

@hoe-jo
hoe-jo force-pushed the joho_examples_separate_modules branch from 51eb379 to 8fe4eb0 Compare July 20, 2026 13:08
@github-actions

Copy link
Copy Markdown

Coverage Report

Coverage report was generated.

Full report can be downloaded from the CI artifacts (expand Artifacts at the bottom of the run).

Overall coverage rate:

lines......: 84.0%
functions......: 52.2%

@LittleHuba LittleHuba left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please check in Otterdog that you do not break the CI 😉

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.

2 participants