Skip to content

Conversation

@ghinks
Copy link
Contributor

@ghinks ghinks commented Dec 24, 2021

Earlier in the year wiby issue 73 was raised and I raised my hand to try to update the documentation. This PR is the result of that work. It is a work in progress, but raising the PR makes this process a collaboration.

rationalize repeated nock code block into a beforeEach clause.
Add some text to describe what wiby is as a prelude
add a simple piece of graphics to see if that helps
- make a simple example with screen shots available
- create some text to give substance to why we have this tool
- add callouts to cli
- add callouts to PR raised
- add wiby clean simple example
- add wiby clean simple example, typo
- wiby clean, grammar
- wiby close-pr, interim push to look at github markup
- wiby close-pr, add image showing PR is closed but branch remains
- wiby validate , readme
- wiby validate , typo in the URI
Add the wiby result command to the readme.
typos
add links to the 5 examples given
add a bit more to explain why we have wiby.
add links to example repos
the array length needs to be checked as an array that is empty will be returned by the reduce and promise all combination
undo checkin on incorrect branch.
Copy link
Member

@bjohansebas bjohansebas left a comment

Choose a reason for hiding this comment

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

LGTM :)

@bjohansebas bjohansebas requested a review from Copilot May 11, 2025 23:39

This comment was marked as resolved.

@bjohansebas bjohansebas merged commit 175848f into pkgjs:main May 11, 2025
9 checks passed
@bjohansebas
Copy link
Member

thanks @ghinks!

@bjohansebas bjohansebas mentioned this pull request May 12, 2025
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