enable e2e/pages-router tests to run in CI

[dario-piotrowicz]

this change renames the e2e script of the e2e/pages-router app to just e2e so that it is included as part of our CI checks, in order for it not to fail one test has been skipped (with an appropriate comment explain why)

---

related to #291

[changeset-bot]

⚠️ No Changeset found

Latest commit: 8e74909

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[pkg-pr-new]

Open in Stackblitz

pnpm add https://pkg.pr.new/@opennextjs/cloudflare@332

commit: 8e74909

[vicb]

LGTM, Thanks Dario!

[vicb]

@dario-piotrowicz I edited to description to rewrite fixes 291 to related to 291 as there is more in there, does that make sense?

[dario-piotrowicz]

@dario-piotrowicz I edited to description to rewrite fixes 291 to related to 291 as there is more in there, does that make sense?

ah yeah sorry I was closing it as all the e2es are not being run, and we do have the extra sub-issues to deal with

yeah given that the remaining ones are sub-issues I guess it does make sense to keep the parent issue open 👍

[vicb]

@dario-piotrowicz I edited to description to rewrite fixes 291 to related to 291 as there is more in there, does that make sense?

ah yeah sorry I was closing it as all the e2es are not being run, and we do have the extra sub-issues to deal with

yeah given that the remaining ones are sub-issues I guess it does make sense to keep the parent issue open 👍

I don't think the build issue has been tackled?

(You can update the parent issue with a status)

Thanks