Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace secrets with random #2106

Merged
merged 5 commits into from
Oct 30, 2023
Merged

Replace secrets with random #2106

merged 5 commits into from
Oct 30, 2023

Conversation

Mandur
Copy link
Contributor

@Mandur Mandur commented Oct 29, 2023

What is being addressed

Randomize unit tests to avoid triggering security scanners

@Mandur Mandur temporarily deployed to CI October 29, 2023 14:28 — with GitHub Actions Inactive
@codecov-commenter
Copy link

codecov-commenter commented Oct 29, 2023

Codecov Report

Merging #2106 (82459eb) into dev (eb5857e) will decrease coverage by 4.76%.
Report is 1 commits behind head on dev.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev    #2106      +/-   ##
==========================================
- Coverage   87.73%   82.97%   -4.76%     
==========================================
  Files         256      256              
  Lines        9309     9309              
  Branches        0     1455    +1455     
==========================================
- Hits         8167     7724     -443     
+ Misses       1142     1140       -2     
- Partials        0      445     +445     

see 130 files with indirect coverage changes

@Mandur Mandur temporarily deployed to CI October 29, 2023 19:38 — with GitHub Actions Inactive
@Mandur Mandur temporarily deployed to CI October 30, 2023 07:51 — with GitHub Actions Inactive
@Mandur Mandur temporarily deployed to CI October 30, 2023 08:13 — with GitHub Actions Inactive
@Mandur Mandur merged commit 1020e5a into dev Oct 30, 2023
7 checks passed
@Mandur Mandur deleted the feature/randomize-tests branch October 30, 2023 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants