Skip to content

feat(core/testers): add hasOption tester#2287

Merged
sdirix merged 2 commits into
eclipsesource:masterfrom
ezyostudio:master
Feb 19, 2024
Merged

feat(core/testers): add hasOption tester#2287
sdirix merged 2 commits into
eclipsesource:masterfrom
ezyostudio:master

Conversation

@zaosoula

@zaosoula zaosoula commented Feb 17, 2024

Copy link
Copy Markdown
Contributor

This PR adds a tester name hasOption allowing to check the presence of an option without specifying its value

@CLAassistant

CLAassistant commented Feb 17, 2024

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@netlify

netlify Bot commented Feb 17, 2024

Copy link
Copy Markdown

Deploy Preview for jsonforms-examples ready!

Name Link
🔨 Latest commit 6cee939
🔍 Latest deploy log https://app.netlify.com/sites/jsonforms-examples/deploys/65d36cd7f44fcf000897a05d
😎 Deploy Preview https://deploy-preview-2287--jsonforms-examples.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.

@sdirix sdirix left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Thanks for the contribution

Comment thread packages/core/src/testers/testers.ts Outdated
Co-authored-by: Stefan Dirix <sdirix@eclipsesource.com>
@coveralls

coveralls commented Feb 19, 2024

Copy link
Copy Markdown

Coverage Status

coverage: 84.861% (+0.07%) from 84.796%
when pulling 6cee939 on ezyostudio:master
into 072caa5 on eclipsesource:master.

@sdirix
sdirix merged commit 51b63f5 into eclipsesource:master Feb 19, 2024
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.

4 participants