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

Remove tests I know... - but they will be elsewhere. #432

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

KarinG
Copy link
Contributor

@KarinG KarinG commented Oct 14, 2023

seamuslee001 added a commit to seamuslee001/com.iatspayments.civicrm that referenced this pull request Oct 20, 2023
@adixon
Copy link
Contributor

adixon commented Apr 4, 2024

It's always good to reduce unnecessary noise and friction, but my hesitation is - isn't more testing better? Isn't it worth fixing instead of removing completely? I'll admit upfront that I'm the poster bad boy of not testing, so if your answer is: "you fix it", I can hear that ...

In terms of the link you posted, it seems like Dave already fixed that specific issue here: 6a721e9

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.

None yet

2 participants