Skip to content

Conversation

LeFrosch
Copy link
Collaborator

@LeFrosch LeFrosch commented Oct 6, 2025

CIdeInfo combines information from two different sources: a) data read from the rule attributes and b) data retrieved from the CcInfo provider.

To keep track what data is retrieved from which source and to provide a clear separation (also for more future fields) this PR separates fields into sub-messages.

Extracted from: #7834

@LeFrosch LeFrosch changed the title Includes Cache: split CIdeInfo into separate messages Include Cache: split CIdeInfo into separate messages Oct 6, 2025
@LeFrosch LeFrosch force-pushed the pull/97f6dcbf708e766b47ea3ba7ff6c356f26b5fd73 branch from 9dfb55c to aa039ab Compare October 6, 2025 10:07
@LeFrosch LeFrosch marked this pull request as ready for review October 6, 2025 10:53
@github-actions github-actions bot added product: CLion CLion plugin product: IntelliJ IntelliJ plugin product: GoLand GoLand plugin awaiting-review Awaiting review from Bazel team on PRs labels Oct 6, 2025
@LeFrosch LeFrosch mentioned this pull request Oct 6, 2025
@LeFrosch LeFrosch merged commit e72e734 into bazelbuild:master Oct 6, 2025
5 checks passed
@LeFrosch LeFrosch deleted the pull/97f6dcbf708e766b47ea3ba7ff6c356f26b5fd73 branch October 6, 2025 11:10
@github-project-automation github-project-automation bot moved this from Untriaged to Done in Bazel IntelliJ Plugin Oct 6, 2025
@github-actions github-actions bot removed the awaiting-review Awaiting review from Bazel team on PRs label Oct 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

product: CLion CLion plugin product: GoLand GoLand plugin product: IntelliJ IntelliJ plugin

Projects

Development

Successfully merging this pull request may close these issues.

4 participants