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 a broken link in cricket-analytics.md #842

Merged
merged 2 commits into from
Apr 3, 2025

Conversation

star1327p
Copy link
Contributor

Copy link

netlify bot commented Mar 22, 2025

Deploy Preview for numpy-org ready!

Name Link
🔨 Latest commit 1648876
🔍 Latest deploy log https://app.netlify.com/sites/numpy-org/deploys/67edfe55db2f6e00083fbffe
😎 Deploy Preview https://deploy-preview-842--numpy-org.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.

Copy link
Member

@InessaPawson InessaPawson left a comment

Choose a reason for hiding this comment

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

Thank you for catching the broken link, @star1327p! The new location of the article appears to be behind a paywall. Is there a copy of it that is open access?

@star1327p
Copy link
Contributor Author

@InessaPawson No, I could not find an open access copy of the article.

@InessaPawson
Copy link
Member

@star1327p Thank you for looking into it! We shouldn’t link to articles that are behind a paywall. I tried to find a similar article with open access with no success. If you find anything, please share it in a comment to this PR so we can discuss it first.

@InessaPawson
Copy link
Member

@star1327p For now, let's remove the broken link. If you come across a similar article/tutorial with open access, please submit a separate PR.

@star1327p
Copy link
Contributor Author

@InessaPawson Sounds good. I have removed the broken link. Thanks!

@InessaPawson InessaPawson changed the title Update a URL in cricket-analytics.md Remove a broken link in cricket-analytics.md Apr 3, 2025
@InessaPawson InessaPawson merged commit 3dae42b into numpy:main Apr 3, 2025
4 checks passed
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.

2 participants