aspect-build / aspect-build/rules_lint

[FR]: Exclude transitioned targets

Open
#409 0 comments 0 reactions 0 assignees View on GitHub
enhancement help wanted
Dominant language
Starlark
Stars
154
Forks
125
Avg merge
3d 21h
Merged PRs (30d)
20

Description

### What is the current behavior?

If a library target is transitioned to a different target platform, the aspect will visit the library more than once.

For example,

```
cc_binary(a)
embedded_target(e1, apps = [a])
embedded_target(e2, apps = [a])
```

context: https://bazelbuild.slack.com/archives/CA31HN1T3/p1728485100416379?thread_ts=1728325365.388799&cid=CA31HN1T3

### Describe the feature

Some way to prevent propagating the aspect across transition boundaries is probably the desired fix.

Contributor guide

Open the contributing guide

Research direction

Start by reproducing the cc_binary and two embedded_target example from the issue, then investigate the aspect's behavior across target-platform transition boundaries. Done means the library target is not visited more than once when shared by transitioned targets, with the intended propagation behavior established and verified.

Written by the indexing model from the issue text.

Assessment

Domain
build-system
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
28/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.