Skip to content
New issue

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

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

Already on GitHub? # to your account

added reference material for expressJS #15436

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

ukor
Copy link

@ukor ukor commented Jan 3, 2025

Summary

ExpressJs is a popular NodeJs library. The Jest documentation used to have good reference material that provided a concise overview of how to start using Jest with ExpressJS. This reference was removed in PR-15270 because of issues with the domain. The reference material domain changed, and it should be included back in the documentation.

Test plan

  • The following commands should be ran in sequence to test these changes
yarn install

cd website

node fetchSupporters.js

yarn start

Screenshot 2025-01-03 at 04 17 54

Copy link

linux-foundation-easycla bot commented Jan 3, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

Copy link

netlify bot commented Jan 3, 2025

Deploy Preview for jestjs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 9db8add
🔍 Latest deploy log https://app.netlify.com/sites/jestjs/deploys/67775f393199830008445041
😎 Deploy Preview https://deploy-preview-15436--jestjs.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.

@mohammednumaan
Copy link
Contributor

Hey! Thanks! This issue was fixed a few days back. For reference view this PR.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants