Skip to content

Add sanitizer reusable workflow#67

Draft
RSingh1511 wants to merge 1 commit intoeclipse-score:mainfrom
RSingh1511:rs/swp-245137
Draft

Add sanitizer reusable workflow#67
RSingh1511 wants to merge 1 commit intoeclipse-score:mainfrom
RSingh1511:rs/swp-245137

Conversation

@RSingh1511
Copy link
Copy Markdown
Contributor

@RSingh1511 RSingh1511 commented Feb 25, 2026

No description provided.

pawelrutkaq
pawelrutkaq requested changes Feb 25, 2026
View reviewed changes
Comment thread .github/workflows/sanitizers.yml
required: false
default: '//tests/...'
type: string
bazelisk_version:
Copy link
Copy Markdown
Contributor

@pawelrutkaq pawelrutkaq Feb 25, 2026

Choose a reason for hiding this comment

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

why this is needed?

Comment thread .github/workflows/sanitizers.yml
disk-cache: ${{ github.workflow }}
repository-cache: true
bazelisk-cache: true
cache-save: ${{ github.event_name == 'push' }}
Copy link
Copy Markdown
Contributor

@pawelrutkaq pawelrutkaq Feb 25, 2026

Choose a reason for hiding this comment

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

also only for main

Comment thread .github/workflows/sanitizers.yml
# SPDX-License-Identifier: Apache-2.0
# *******************************************************************************

name: Sanitizers (Reusable)
Copy link
Copy Markdown
Contributor

@pawelrutkaq pawelrutkaq Feb 25, 2026

Choose a reason for hiding this comment

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

C++ Sanitizers?

Comment thread .github/workflows/sanitizers.yml
on:
workflow_call:
inputs:
sanitizer_configs:
Copy link
Copy Markdown
Contributor

@pawelrutkaq pawelrutkaq Feb 25, 2026

Choose a reason for hiding this comment

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

shoudn't that be just bazel config ? Then user ofc will provie them.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 1, 2026

Thanks for the pull request. This thread has been quiet for 30 days, so we are marking it as stale for now. Please take a quick look and let us know whether it is still up to date, still relevant, needs review, or is ready to merge. Any new activity will remove the stale label automatically. If nothing changes in the next 10 days, we will close it to keep the backlog current.

@github-actions github-actions Bot added the stale label Apr 1, 2026
@4og 4og removed the stale label Apr 7, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 8, 2026

Thanks for the pull request. This thread has been quiet for 30 days, so we are marking it as stale for now. Please take a quick look and let us know whether it is still up to date, still relevant, needs review, or is ready to merge. Any new activity will remove the stale label automatically. If nothing changes in the next 10 days, we will close it to keep the backlog current.

@github-actions github-actions Bot added the stale label May 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pawelrutkaq pawelrutkaq pawelrutkaq requested changes

@AlexanderLanin AlexanderLanin Awaiting requested review from AlexanderLanin AlexanderLanin will be requested when the pull request is marked ready for review AlexanderLanin is a code owner

@MaximilianSoerenPollak MaximilianSoerenPollak Awaiting requested review from MaximilianSoerenPollak MaximilianSoerenPollak will be requested when the pull request is marked ready for review MaximilianSoerenPollak is a code owner

@dcalavrezo-qorix dcalavrezo-qorix Awaiting requested review from dcalavrezo-qorix dcalavrezo-qorix will be requested when the pull request is marked ready for review dcalavrezo-qorix is a code owner

@nradakovic nradakovic Awaiting requested review from nradakovic nradakovic will be requested when the pull request is marked ready for review nradakovic is a code owner

@PiotrKorkus PiotrKorkus Awaiting requested review from PiotrKorkus PiotrKorkus will be requested when the pull request is marked ready for review PiotrKorkus is a code owner

@opajonk opajonk Awaiting requested review from opajonk opajonk will be requested when the pull request is marked ready for review opajonk is a code owner

@lurtz lurtz Awaiting requested review from lurtz lurtz will be requested when the pull request is marked ready for review lurtz is a code owner

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

stale

Projects

INF - Infrastructure Community
Status: Backlog

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@RSingh1511 @pawelrutkaq @4og