Skip to content

Navigation Menu

Appearance settings

View all features
- BY COMPANY SIZE
  Enterprises
  Small and medium teams
  Startups
  Nonprofits
- BY USE CASE
  App Modernization
  DevSecOps
  DevOps
  CI/CD
  View all use cases
- BY INDUSTRY
  Healthcare
  Financial services
  Manufacturing
  Government
  View all industries
View all solutions
- EXPLORE BY TOPIC
  AI
  Software Development
  DevOps
  Security
  View all topics
- EXPLORE BY TYPE
  Customer stories
  Events & webinars
  Ebooks & reports
  Business insights
  GitHub Skills
- SUPPORT & SERVICES
  Documentation
  Customer support
  Community forum
  Trust center
  Partners
- COMMUNITY
  GitHub SponsorsFund open source developers
- PROGRAMS
  Security Lab
  Maintainer Community
  Accelerator
  Archive Program
- REPOSITORIES
  Topics
  Trending
  Collections
- ENTERPRISE SOLUTIONS
  Enterprise platformAI-powered developer platform
- AVAILABLE ADD-ONS
  GitHub Advanced SecurityEnterprise-grade security features
  Copilot for BusinessEnterprise-grade AI features
  Premium SupportEnterprise-grade 24/7 support
Pricing

Search code, repositories, users, issues, pull requests...

Search

Clear

Search syntax tips

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Name

Query

To see all available qualifiers, see our documentation.

Appearance settings

You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session.

Dismiss alert

paypal / paypal-checkout-components Public

Notifications You must be signed in to change notification settings
Fork 601
Star 1.3k

Code
Pull requests 10
Actions
Security
Insights

Additional navigation options

Code
Pull requests
Actions
Security
Insights

Skip screenshot tests in CI#2348

Merged

nbierdeman merged 59 commits intov4paypal/paypal-checkout-components:v4from

fix-v4-cipaypal/paypal-checkout-components:fix-v4-ciCopy head branch name to clipboard

Mar 6, 2024

Conversation Commits59 (59)Checks Files changed

Merged

Skip screenshot tests in CI#2348
nbierdeman merged 59 commits intov4from
fix-v4-ci

Commits

Commits on Feb 28, 2024

skip webpack
nbierdeman
committed
ci failed -> webpack is cool again
nbierdeman
committed
skip jest screenshot tests
nbierdeman
committed
only run webpack and jest screenshot tests
nbierdeman
committed
yep, jest screenshot tests break ci
nbierdeman
committed
let's see what happens when grabthar-release is upgraded
nbierdeman
committed
seems like we're stuck with grabthar-release v1.0.69
nbierdeman
committed
what happens when we remove an event listener?
nbierdeman
committed
the event listener was not the culprit
nbierdeman
committed
what if we run jest without the env option?
nbierdeman
committed
nah, the env option was fine
nbierdeman
committed
try ci option
nbierdeman
committed
nah, the ci option didn't work
nbierdeman
committed

Commits on Feb 29, 2024

add --help to jest
nbierdeman
committed
yep, CI executes jest --help
nbierdeman
committed
setup new gh actions task for jest screenshots
nbierdeman
committed
remove flow since it keeps failing now that screenshots are a separate task
nbierdeman
committed
the separate screenshot task still hung
nbierdeman
committed
add screenshot test logs
nbierdeman
committed
add more screenshot logs
nbierdeman
committed
even more screenshot logs
nbierdeman
committed

Commits on Mar 1, 2024

start from the top
nbierdeman
committed
remove npm run buttondiff from 'test'
nbierdeman
committed
skip flow
nbierdeman
committed
remove condition
nbierdeman
committed
fix flow
wsbrunson
committed
break up tasks in main.yml
wsbrunson
committed
fix yml
wsbrunson
committed
remove matrix
wsbrunson
committed
whoops lol
wsbrunson
committed
run screenshot tests in separate action
nbierdeman
committed

Commits on Mar 5, 2024

try running a single screenshot test in ci
nbierdeman
committed
double timeout time
nbierdeman
committed
nah, doubling the jest timeout didn't work
nbierdeman
committed
add setupBrowserPage log
nbierdeman
committed
use async/await in beforeAll
nbierdeman
committed
remove beforeAll
nbierdeman
committed
add beforeAll
nbierdeman
committed
remove iffy IIFE
nbierdeman
committed

Commits on Mar 6, 2024

add iffy IIFE back in
nbierdeman
committed
more setupBrowserPage logs
nbierdeman
committed
more logs
nbierdeman
committed
more logs
nbierdeman
committed
remove setupBrowserPage for...of loop
nbierdeman
committed
remove setupBrowserPage page.evaluate()
nbierdeman
committed
remove async/await from beforeAll
nbierdeman
committed
revert beforeAll to original state
nbierdeman
committed
revert screenshot tests to original state
nbierdeman
committed
skip screenshot tests in ci
nbierdeman
committed
remove config
nbierdeman
committed
add more scripts to test script
nbierdeman
committed
add husky
nbierdeman
committed
example comment
nbierdeman
committed
add husky pre-push
nbierdeman
committed
remove husky object
nbierdeman
committed
remove test comment
nbierdeman
committed
run screenshot tests in pre-push hook
nbierdeman
committed
skip pre-push hook in ci
nbierdeman
committed
use exit code from npm run buttondiff
nbierdeman
committed

Footer

© 2026 GitHub, Inc.

Footer navigation

Terms
Privacy
Security
Status
Community
Docs
Contact

You can’t perform that action at this time.