Commit Graph

7 Commits

Author SHA1 Message Date
Arpit Mohan
2650ea161d
ci: Updating actions/checkout to v4 and defaulting to fetch-depth 1 instead of 0 (#29281)
<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

- **Chores**
- Updated GitHub Actions workflows to use `actions/checkout@v4` for
improved performance and reliability.
- Removed `fetch-depth` parameter to simplify checkout steps across
various workflows.
  - Standardized quote usage for consistency in workflow files.

- **Documentation**
- Adjusted formatting and descriptions in workflow files for better
clarity and readability.

- **Refactor**
- Aligned multiple workflow files to follow a consistent structure and
naming convention.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

fetch-depth 0 causes the Github workflow to checkout the entire Git
history. This is not required. We only need to check out the head of the
commit. By default, actions/checkout has fetch-depth=1, hence removing
it from the workflow completely for simplicity.
2023-12-05 13:44:43 +05:30
Shrikant Sharat Kandula
1504254d84
ci: Fix issue title injections in workflow 2023-05-02 11:02:43 +05:30
Arun
693aa75e79
fix: Security Vulnerability in GitHub Workflows (#22888)
"github.event.issue.title" is potentially untrusted. avoid using it
directly in inline scripts. instead, pass it through an environment
variable. see
https://docs.github.com/en/actions/security-guides/security-hardening-for-github-actions
for more details.
2023-05-02 10:59:34 +05:30
Aishwarya-U-R
14c63fc7f3
Update CypressAddKnownfailedtests.yml 2023-02-24 13:32:27 +05:30
Aishwarya-U-R
7ab395979f
Added bug type to include reopened 2023-02-24 13:30:43 +05:30
Aishwarya-U-R
9976ae380d
Updated lable CI impacted 2023-02-24 13:27:32 +05:30
yatinappsmith
876cd428f9
ci: Comment failing tests to PR (#18450)
## Description

Comment failing tests to PR

Fixes # 



## Type of change

- New feature (non-breaking change which adds functionality)


## How Has This Been Tested?
Code review. Tested in local fork

### Test Plan
> Add Testsmith test cases links that relate to this PR

### Issues raised during DP testing
> Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)


## Checklist:
### Dev activity
- [X] My code follows the style guidelines of this project
- [X] I have performed a self-review of my own code
- [X] I have commented my code, particularly in hard-to-understand areas
- [X] I have made corresponding changes to the documentation
- [X] My changes generate no new warnings
- [X] I have added tests that prove my fix is effective or that my feature works
- [X] New and existing unit tests pass locally with my changes
- [X] PR is being merged under a feature flag


### QA activity:
- [ ] Test plan has been approved by relevant developers
- [ ] Test plan has been peer reviewed by QA
- [ ] Cypress test cases have been added and approved by either SDET or manual QA
- [ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
- [ ] Added Test Plan Approved label after reveiwing all Cypress test
2022-12-19 16:23:39 +05:30