[2.x] add toBeEmailAddress() expectation

[JonPurvis]

What:

• Bug Fix
• New Feature

Description:

This PR adds a toBeEmailAddress() Expectation, to validate a value is an email address.

<?php

it ('ensures the email of the user is actually an email', function () {
    expect($user->email)->toBeEmailAddress();
});

[devajmeireles]

Hey, @JonPurvis ! Thanks for your PR.

Although it seems useful, I would disagree with this PR. 👎🏻

I know you may have invested time on this PR, I'm sorry about that, but this PR sets a huge precedent for other validations, which differs from Pest's focus at the moment.

Feel free to submit other PRs based on your suggestion whenever you like.

[JonPurvis]

Hey @devajmeireles 👋

Thanks for taking a look! I'm not sure I follow though. This PR is pretty similar to the isURL() Expectation, which was merged and released sometime last week. The only difference is, this is checking email addresses rather than URLs.

Please could you elaborate on:

this PR sets a huge precedent for other validations

Thanks!

[JonPurvis]

Hey @devajmeireles

Just a polite bump to check in on your thoughts on this PR 😄

[JonPurvis]

Hey @devajmeireles

Another bump on this please. If you could find the time, can you please elaborate on your points made last month?

Thanks

[devajmeireles]

Hey @devajmeireles

Another bump on this please. If you could find the time, can you please elaborate on your points made last month?

Thanks

Nevermind.

[nunomaduro]

No plans for this.