Skip to content

first 3 complete#3

Open
jbok4 wants to merge 16 commits into
rails-camp:masterfrom
jbok4:master
Open

first 3 complete#3
jbok4 wants to merge 16 commits into
rails-camp:masterfrom
jbok4:master

Conversation

@jbok4

@jbok4 jbok4 commented Mar 12, 2017

Copy link
Copy Markdown

No description provided.

@kotp

kotp commented Mar 23, 2017

Copy link
Copy Markdown

This looks like a lot more than the first three. ;)

@jbok4

jbok4 commented Mar 23, 2017 via email

Copy link
Copy Markdown
Author

@kotp

kotp commented Mar 23, 2017

Copy link
Copy Markdown

Near the top of this Pull Request there are three tabs... Conversation 2 | Commits 16 | Files changed 19

Every time you push to this branch you are updating this pull request.

That isn't really a problem, but now the title doesn't match the pull request.

@jordanhudgens

Copy link
Copy Markdown
Contributor

It's fine, that's just the reason I haven't been merging them in. That would override the solutions that the posts point to. Nice work on them though!

@jbok4

jbok4 commented Mar 23, 2017 via email

Copy link
Copy Markdown
Author

@kotp

kotp commented Mar 23, 2017

Copy link
Copy Markdown

Or create a new branch for each exercise, or whatever the organization is you want to do. Not an annoyance, was just getting increasingly out of sync for the name of this PR.

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.

3 participants