Changed "Active Prodcuts" to "Active Products" on Releeph Product Query dropdown.
(Incidentally, this is my first diff on Phabricator proper, so I hope I got it right. I'm used to SVN, not Git.)
Details
Details
- Reviewers
epriestley chad - Group Reviewers
Blessed Reviewers - Commits
- Restricted Diffusion Commit
rP1f54b95324eb: Fixed typo on Releeph product query dropdown. - Required Signatures
L28 Phacility Individual Contributor License Agreement
Make sure the change is there and didn't bork anything else.
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
Follow the instructions in the lint message to clear lint. They are your fault, and the very first part of the message explains how to resolve them.
Comment Actions
Sorry, get rid of the shebang. The shebang warning is because the file is +x in your environment for some reason; it should not be. It is not +x in the repository.
Anyway, this is close enough. I'll fix it in the pull.
Comment Actions
(I'm near totally unfamiliar with Git. I always use SVN.) I ran arc land 'master' as instructed at the top, and it failed with
You can not land a branch onto itself -- you are trying to land 'master' onto 'master'. For more information on how to push changes, see 'Pushing and Closing Revisions' in 'Arcanist User Guide: arc diff' in the documentation. You may be able to 'arc amend' instead.
So, I ran arc amend, and that said
Amending commit message to reflect revision D12904: Fixed typo on Releeph product query dropdown..
...but did not apparently commit. Repeated both, same results. The Arcanist User Guide tells me nothing additional. I'm lost.
Comment Actions
You don't have commit rights to the repository. I've pulled the change, I just had a few other things come up.
Comment Actions
Ah, okay, well that explains that. I'm still getting my bearings around here, as my own workflows are muscle memory. Thank you!