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

docs: remove section "More Information" #1303

Merged

Conversation

MikeMcC399
Copy link
Collaborator

Issue

The README > More Information section is made up of three links:

  1. blog post Drastically Simplify Testing on CI with Cypress GitHub Action from Nov 2019
  2. blog post Test the Preview Vercel Deploys from Aug 2020
  3. link to GitHub Actions documentation Creating actions which now redirects to a renamed section Sharing automations.

The two blog posts are based on now-unsupported legacy versions.

The remaining link to a section of the GitHub Actions documentation would be out of context left on its own. The README already generically links to the GitHub Actions documentation in its opening paragraph. This covers the linked subsection Sharing automations, so the link in the README > More Information section is effectively redundant.

Change

Remove the outdated section More Information from the README document.

@MikeMcC399 MikeMcC399 added the documentation Improvements or additions to documentation label Nov 11, 2024
@MikeMcC399 MikeMcC399 self-assigned this Nov 11, 2024
@cypress-app-bot
Copy link

@MikeMcC399 MikeMcC399 marked this pull request as ready for review November 11, 2024 10:20
@jennifer-shehane
Copy link
Member

@MikeMcC399 Thanks. I'm going to be out of town for much of November, but I don't want these PRs to be lost among the team when they're on these repos that you triage. Are you able to tag @cypress-io/end-to-end as a reviewer? That would tag our team - they're taking turns triaging while I'm out.

@jennifer-shehane jennifer-shehane merged commit ff35f50 into cypress-io:master Nov 12, 2024
77 checks passed
@MikeMcC399 MikeMcC399 deleted the remove/more-information branch November 12, 2024 16:40
@MikeMcC399
Copy link
Collaborator Author

@jennifer-shehane

I'm going to be out of town for much of November, but I don't want these PRs to be lost among the team when they're on these repos that you triage. Are you able to tag @cypress-io/end-to-end as a reviewer? That would tag our team - they're taking turns triaging while I'm out.

Thank you for letting me know about your planned absence. No, I don't have visibility on @cypress-io/end-to-end from outside.

@jennifer-shehane
Copy link
Member

@MikeMcC399 Hmm, that's annoying. Could you add @AtofStryker instead then? Thanks.

@MikeMcC399
Copy link
Collaborator Author

@jennifer-shehane

Hmm, that's annoying. Could you add @AtofStryker instead then?

Sure, I'm happy to do that. BTW I'm also on Discord with the same username if you ever need to DM me.

aayushshah15 pushed a commit to useblacksmith/cypress-io that referenced this pull request Dec 3, 2024
Copy link

github-actions bot commented Dec 9, 2024

🎉 This PR is included in version 6.7.8 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants