Skip to content

fix(README): Update README.md #142

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

Merged
merged 2 commits into from
May 27, 2024
Merged

fix(README): Update README.md #142

merged 2 commits into from
May 27, 2024

Conversation

fhlavac
Copy link
Collaborator

@fhlavac fhlavac commented May 23, 2024

Updated readme scripts

@patternfly-build
Copy link

patternfly-build commented May 23, 2024

README.md Outdated
- run npm run build:docs followed by npm run serve:docs, then run npm run test:a11y in a new terminal window to run our accessibility tests for the components. Once the accessibility tests have finished running you can run
- npm run serve:a11y to locally view the generated report.
- run `npm run build:docs` followed by `npm run serve:docs`, then run `npm run test:a11y` in a new terminal window to run our accessibility tests for the components. Once the accessibility tests have finished running you can run
- run `npm run serve:a11y` to locally view the generated report.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think you have an extra "run" at the beginning of this line.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Good catch, thank you 🙂

@fhlavac fhlavac merged commit e877107 into main May 27, 2024
@fhlavac fhlavac deleted the fhlavac-patch-3 branch May 27, 2024 14:04
Copy link

🎉 This PR is included in version 5.2.0-prerelease.4 🎉

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants