docs(cli): add dev docs and update#1046
docs(cli): add dev docs and update#1046lwasser wants to merge 1 commit intoall-contributors:mainfrom
Conversation
✅ Deploy Preview for all-contributors ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
ok @JimMadge, I'd suggest for my PR"s in the future, don't hit the update branch button because I'll rebase locally. Once a merge commit is in the mix, things get very messy and difficult to clean up. I prefer the rebase as it's cleaner. But for other contributors, we can update the branch right before we merge, and it should be ok. Maybe we can read about this more when you squash-merge, and there's a merge commit instead of a rebase. I don't know how that works. |
|
When I run the linter locally, the README file is failing. But I didn't touch the README file, so I'd rather that get fixed outside of this pr which is just focused on CLI docs. We could turn this into an issue if needed. |
Oh, sorry for giving you trouble. I didn't use the update branch button. It wasn't available because there were conflicts with the target branch. I did a merge manually (mainly because I prefer merges and don't like having to fix conflicts multiple times in a rebase 😆). Is there a problem with rebasing a branch after a merge commit? Maybe git has trouble knowing which parent to use? Maybe you can squash the branch then rebase? In any case, sorry, I didn't foresee that. Does this branch include the changes from the comments in the PR? were there any left unresolved? |
|
Regarding semantic line breaks, we don't say anything about expecting or enforcing that. But the existing documentation is in that style, so I would say it is better for consistency. I won't block this PR because of that though, so I suppose that is a soft ask from me 😆. |
JimMadge
left a comment
There was a problem hiding this comment.
Assuming this is as the last PR with the suggestions, good 👍.
There are some linting problems with the new cli development file. Once those are resolved I think we can merge. The other questions aren't important enough to block anything.
I can't reproduce this locally. What was the command you ran and package versions? |
This is a reado of #1032 which i'll close
Checklist:
Bot Usage