Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Develop #11

Open
wants to merge 20 commits into
base: uiposition
Choose a base branch
from
Open

Develop #11

wants to merge 20 commits into from

Conversation

bradrich
Copy link

@bradrich bradrich commented Oct 4, 2015

Submission of the requested coding challenge. Please review README.md file located in single-page-app-exercise to understand setup, viewing and testing.

You might have noticed the closing of the pull request of my master to your master. I thought better of it since it was not in the mind of Git Flow best practices.

@spencerhrob
Copy link
Contributor

I can't quite tell why you closed #10 and opened #11, but if you're just
updating your code, you can simply keep your pull request open and continue
to push new code to your branch. GitHub will automatically update your pull
request with your new code.
On Oct 4, 2015 2:50 PM, "Brad Richardson" [email protected] wrote:

Submission of the requested coding challenge. Please review README.md file
located in single-page-app-exercise to understand setup, viewing and
testing.

You might have noticed the closing of the pull request of my master to
your master. I thought better of it since it was not in the mind of Git

Flow best practices.

You can view, comment on, or merge this pull request online at:

#11
Commit Summary

  • Merge pull request Uiposition #5 from siostechcorp/uiposition
  • Update README
  • Again, generator
  • Added data.json
  • Removed extra stuff, ready to go
  • Fixed missing timestamps, fixed ports in README
  • Travis' questions
  • Merge pull request Update README #7 from siostechcorp/readme
  • Merge pull request Generator2 #8 from siostechcorp/generator2
  • Merge pull request Travis' questions #9 from siostechcorp/questions
  • Spelling fixes
  • Initial commit.
  • Working on unit testing.
  • * Fix - Fixed a date issue with Internet Explorer.
  • Working on initial build.
  • Finalize testing.
  • Finalizing build.
  • Fixed an error in the unit tests.
  • Readme change.
  • Readme change.

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#11.

@bradrich
Copy link
Author

bradrich commented Oct 4, 2015

I understand and am familiar with that process. I was not adding more code. I was simply concerned with the request of my master branch being pulled into your master branch. I generally like to follow Git Flow best practices and wanted to change it to where it was my develop branch being pulled into a branch besides your master. Please excuse the traffic.

@bradrich
Copy link
Author

bradrich commented Oct 4, 2015

It has been bothering me since yesterday and my OCD won't let me ignore it, LOL.

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.

4 participants