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
Contributor

No description provided.

required: false
default: '//tests/...'
type: string
bazelisk_version:
Copy link
Contributor

Choose a reason for hiding this comment

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

why this is needed?

disk-cache: ${{ github.workflow }}
repository-cache: true
bazelisk-cache: true
cache-save: ${{ github.event_name == 'push' }}
Copy link
Contributor

Choose a reason for hiding this comment

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

also only for main

# SPDX-License-Identifier: Apache-2.0
# *******************************************************************************

name: Sanitizers (Reusable)
Copy link
Contributor

Choose a reason for hiding this comment

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

C++ Sanitizers?

on:
workflow_call:
inputs:
sanitizer_configs:
Copy link
Contributor

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Draft

Development

Successfully merging this pull request may close these issues.

2 participants