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.

Sign up for GitHub

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

Jump to bottom

Reduce the cypress GitHub runner utilization to 4 #8987

Merged
merged 16 commits into from
Nov 4, 2024
Merged

Reduce the cypress GitHub runner utilization to 4 #8987

merged 16 commits into from
Nov 4, 2024

Conversation

nihal467
Copy link
Member

@nihal467 nihal467 commented Nov 3, 2024 •
edited
Loading

Proposed Changes

  • reduce the runner count, which bring the utilisation from 56 Minutes to 35 Minutes
  • individual runner entire setup will be completed in 8 minute 36 second rather than 6 minutes 36 seconds
  • the entire test file completion will be increased by 1 minute

image

@ohcnetwork/care-fe-code-reviewers

Merge Checklist

  • Add specs that demonstrate bug / test a new feature.
  • Update product documentation.
  • Ensure that UI text is kept in I18n files.
  • Prep screenshot or demo video for changelog entry, and attach it to issue.
  • Request for Peer Reviews
  • Completion of QA

@cloudflare-workers-and-pages Cloudflare Workers and Pages
Copy link

cloudflare-workers-and-pages bot commented Nov 3, 2024 •
edited
Loading

Deploying care-fe with  Cloudflare Pages  Cloudflare Pages

Latest commit: e64e8af
Status:⚡️  Build in progress...

View logs

@netlify Netlify
Copy link

netlify bot commented Nov 3, 2024 •
edited
Loading

✅ Deploy Preview for care-ohc ready!

Name Link
🔨 Latest commit e64e8af
🔍 Latest deploy log https://app.netlify.com/sites/care-ohc/deploys/67286f104a549e00084ca4c6
😎 Deploy Preview https://deploy-preview-8987--care-ohc.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@cypress Cypress
Copy link

cypress bot commented Nov 3, 2024 •
edited
Loading

CARE    Run #3797

Run Properties:  status check passed Passed #3797  •  git commit e4018d2f63: Cypress resource
Project CARE
Branch Review cypress-resource
Run status status check passed Passed #3797
Run duration 04m 48s
Commit git commit e4018d2f63: Cypress resource
Committer Mohammed Nihal
View all properties for this run ↗︎
Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 125
View all changes introduced in this branch ↗︎

@nihal467 nihal467 marked this pull request as ready for review November 3, 2024 21:16
@nihal467 nihal467 requested a review from a team as a code owner November 3, 2024 21:16
@nihal467 nihal467 changed the title Cypress resource Reduce the github runner from 8 to 4 Nov 3, 2024
@nihal467 nihal467 changed the title Reduce the github runner from 8 to 4 Reduce the cypress GitHub runner utilization to 4 Nov 3, 2024
@nihal467 nihal467 self-assigned this Nov 3, 2024
@rithviknishad rithviknishad merged commit 5877d9b into develop Nov 4, 2024
12 of 16 checks passed
@rithviknishad rithviknishad deleted the cypress-resource branch November 4, 2024 06:52
@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 4, 2024

@nihal467 @nihal467 Your efforts have helped advance digital healthcare and TeleICU systems. 🚀 Thank you for taking the time out to make CARE better. We hope you continue to innovate and contribute; your impact is immense! 🙌

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

@rithviknishad rithviknishad rithviknishad approved these changes

Assignees

@nihal467 nihal467

Labels
None yet
Projects
Care
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nihal467 @rithviknishad