Skip to content

Develop mathquill#921

Closed
mgage wants to merge 3 commits into
openwebwork:developfrom
mgage:develop_mathquill
Closed

Develop mathquill#921
mgage wants to merge 3 commits into
openwebwork:developfrom
mgage:develop_mathquill

Conversation

@mgage

@mgage mgage commented Feb 1, 2019

Copy link
Copy Markdown
Member

This seems be a cleaner version of the mathquill PR #920 . It only changes 28 files and doesn't have the massive 150 file commit. I cherry picked the essential macquill changes directly into a fresh version of develop and then reconciled changes in 10 or so files.

@drdrew42

drdrew42 commented Feb 1, 2019

Copy link
Copy Markdown
Member

@mgage I have some further improvements to this branch - how should I go about this?

@mgage

mgage commented Feb 1, 2019 via email

Copy link
Copy Markdown
Member Author

@drdrew42

drdrew42 commented Feb 1, 2019

Copy link
Copy Markdown
Member

Okay, I'll try following this - but I've got one lingering issue that I want to clear up before I push this next collection of updates. The only updates will be to mathquill.js and mqeditor.js

@dpvc

dpvc commented Feb 1, 2019

Copy link
Copy Markdown
Member

@mgage, I don't think this is quite the right procedure. Andrew may not have your fork as a remote for his clone, and so he will have to set that up first. Then the only thing he has to do it check out your copy of the branch and it will create a branch automatically for him. Finally, your process would merge into his copy of develop, rather than the fresh_develop branch.

So I think what he should do is something more like

git remote add https://github.com/mgage/webwork2.git mgage
git fetch mgage
git checkout mgage/develop_mathquill

this should get him a develop_mathquill branch in his own fork that he can commit to and push. Then he can either make a pull request to the main repo from that, or a PR to your fork if you would like to make the final PR.

@mgage

mgage commented Feb 1, 2019

Copy link
Copy Markdown
Member Author

@dpvc -- you are right -- the checkout develop was a typo which I've corrected to checkout fresh_develop in case someone else reads it. I thought checking out develop first was not necessary but it has become a habit of mine to make sure I'm on the right branch and it is pointed at a known target (origin/develop).

Andrew go ahead and make a PR to the main repo and once that is done I will close the other PR. This is basically your PR there is no need for me to be in the middle of it.

@mgage

mgage commented Feb 1, 2019

Copy link
Copy Markdown
Member Author

replaced by PR #922

@mgage mgage closed this Feb 1, 2019
@mgage mgage deleted the develop_mathquill branch March 11, 2019 19:54
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