salarkhan / git-pear Goto Github PK

bullshit.

in this case, we should fallback to constructing name using the first half of their email address

AND we should echo a message asking them to update their email address on github

it is time.

Or some other word that more accurately reflects the ending of a session.

We are checking for git directory before we are checking for uninstall. it is likely that users may want to uninstall pear without being in a git directory.

brew pear vs brew git-pear

relevant links

• https://github.com/Homebrew/homebrew/wiki/Formula-Cookbook
• http://apple.stackexchange.com/questions/97065/how-do-i-write-a-homebrew-recipe

Add currently adds a user and adds a ghost user. adding the same user twice should probably not be attached to this function

I signed out of pear and i still had a user in the local config.

I don't think we are scrubbing the user key
i ran
$ pear
and still had this in .git/config

We should be able to append what we need to the pre-commit hook with our-pre-commit >> .git/pre-commit and i think we can remove it with ``our-pre-commit << .git/pre-commit`

I think we are gonna need an if clause to catch the shebang though... or we can just keep adding it forever as it is a comment but that seems stupid

• update screenshots
• git log --full or whatever. whatever shows how the committers AND authors are switching
• solo
• GIT_API_TOKEN instructions

Here are the 2 options I see:

• pear [email protected] [email protected]
• pear add [email protected] [email protected]

The latter will allow us to maintain the current architecture in pear, while the former will require some shenanigans around lines 26-32 in pear.

Something along the lines of running a regex against the argument to check if it's an email && firing pear add $command? Seems kinda dumb..

pear [email protected] followed by pear [email protected]
will yield both of us. Also typos will stay around forever, the check for @ symbols should run uninstall.

Here's the error:

And here's why:

We need to give a default username. Maybe just a blank space will do?

testing will require us to commit often and to want to remove the pieces that are under construction.

We should build the uninstall feature sooner than later to lower the burden of testing

Assuming the only commands the end user should ever need are 'pear', 'pear email email', and 'pear uninstall' --

Are we gonna have a file who's job it is to handle user input && invoke the respective files? Cause I was kinda confused about how file invocation within the application is supposed to work

As referenced in #44, we should replace "Pear" with the Github usernames ( or real names ) of each pair member.

I imagine this will involve us making an API call to Github, so we should prob look into getting that API key as well.

This is the final step before MVP. WOOP.

If we can't control where the user installs pear, how can we know how to invoke it?

seems like a likely use case on our workstations

typing just pear
should clear the user list and resume normal functionality to the program
this should be followed by the usage message

the usage message should explain that this happens if we implement this

We should ensure that users don't pear add [email protected] if that email is already in the team.txt

We should pass the path as argument instead of exporting PEAR_WORKING_DIR.
I think we can simply add it as another argument to uninstall and cycle... which i believe are the only two programs who need to know that location outside of the actual pair app

"man, I'm super proud of what we've done here.

I would welcome all derogatory comments, since all they'll do is make this project sicker "

have at it plx @quackingduck @mweitzel

please note this is intended for a brew install so at the moment for testing as written the path must be amended to

PATH="$PATH:."

pear calls the command pear for functionality. If it's not in the path life is unhappy

it also probably needs an API key as i assume we will run into the same rate limiting problem we currently experience with iam weare

i don't think me or salar are opposed to some DBC branding in the read me or usage for shits and giggles. Wherever it goes i had fun writing this.

Maybe this is a good thing for the students lololol.

Not really sure how pear is borking commit -m though..

homebrew create usr/local/bin (and probably own it for the user too)
therefore our install will fail without home-brew

post-committing doesn't cover this case as currently written