Conversation
|
We can continue with this, once #486 is merged |
There was a problem hiding this comment.
Pull request overview
This PR updates the README documentation by fixing broken links and updating badges to reflect current infrastructure. The changes remove outdated Slack and CircleCI references, replacing them with GitHub Actions badges and forum links.
Key changes:
- Replaces CircleCI and Slack badges with GitHub Actions workflow badge
- Updates internal project links to point to correct repository paths
- Clarifies Meteor project setup instructions regarding the
--blazeflag
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ## Quick Start | ||
|
|
||
| Blaze is a [Meteor](http://meteor.com/)-only package for now. Soon we will have Blaze on npm so you can use it in your stack. | ||
| Blaze is a [Meteor](http://meteor.com/)-only package for now. In the future, we intend have Blaze on npm so you can use it in your stack. |
There was a problem hiding this comment.
The sentence has a grammatical error. It should read "we intend to have" instead of "we intend have" (missing "to").
| </a> | ||
| <a href="https://circleci.com/gh/meteor/blaze" target="_blank"> | ||
| <img src="https://circleci.com/gh/meteor/blaze.svg?style=shield"> | ||
| <a href="#Blaze Tests" href="https://github.com/meteor/blaze/actions/workflows/blaze-tests.yml"> |
There was a problem hiding this comment.
The anchor tag has two href attributes. The first one "href='#Blaze Tests'" should be removed or changed to a different attribute like "title" or "aria-label". HTML elements should only have one href attribute.
1 participant
I found the README contains a few broken links and confusing information. Additionally the badges are updated using the GitHub actions status.