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

Error message from git update is not helpful. #41

Closed
ma2ciek opened this issue Feb 8, 2017 · 2 comments
Closed

Error message from git update is not helpful. #41

ma2ciek opened this issue Feb 8, 2017 · 2 comments

Comments

@ma2ciek
Copy link
Contributor

ma2ciek commented Feb 8, 2017

I've got following error after git update:

(node:11920) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 2): TypeError: msg.trim is not a function

I checked packages and one of them was on the other branch that needs to merge after git pull

 #  @ckeditor/ckeditor5-utils                                            16/18 (89%) 
Your configuration specifies to merge with the ref 'refs/heads/t/116'
from the remote, but no such ref was fetched.

So the error from git update could be similar to provide a useful insight into the problem.

@pomek
Copy link
Member

pomek commented Feb 8, 2017

Would be nice to have some steps to reproduce. It speeds up the fix.

@pomek pomek added the type:bug label Feb 8, 2017
@pomek pomek self-assigned this Feb 8, 2017
@ma2ciek
Copy link
Contributor Author

ma2ciek commented Feb 8, 2017

Aw, I read the message once again and it was particularly my fault to pull already not-existing branch.

mgit.json:

{
  "packages": "packages/",
  "dependencies": {
    "@ckeditor/ckeditor5-autoformat": "ckeditor/ckeditor5-autoformat",
    "@ckeditor/ckeditor5-basic-styles": "ckeditor/ckeditor5-basic-styles",
    "@ckeditor/ckeditor5-clipboard": "ckeditor/ckeditor5-clipboard",
    "@ckeditor/ckeditor5-core": "ckeditor/ckeditor5-core",
    "@ckeditor/ckeditor5-editor-classic": "ckeditor/ckeditor5-editor-classic",
    "@ckeditor/ckeditor5-engine": "ckeditor/ckeditor5-engine",
    "@ckeditor/ckeditor5-enter": "ckeditor/ckeditor5-enter",
    "@ckeditor/ckeditor5-heading": "ckeditor/ckeditor5-heading",
    "@ckeditor/ckeditor5-image": "ckeditor/ckeditor5-image",
    "@ckeditor/ckeditor5-link": "ckeditor/ckeditor5-link",
    "@ckeditor/ckeditor5-list": "ckeditor/ckeditor5-list",
    "@ckeditor/ckeditor5-markdown-gfm": "ckeditor/ckeditor5-markdown-gfm",
    "@ckeditor/ckeditor5-paragraph": "ckeditor/ckeditor5-paragraph",
    "@ckeditor/ckeditor5-theme-lark": "ckeditor/ckeditor5-theme-lark",
    "@ckeditor/ckeditor5-typing": "ckeditor/ckeditor5-typing",
    "@ckeditor/ckeditor5-ui": "ckeditor/ckeditor5-ui",
    "@ckeditor/ckeditor5-undo": "ckeditor/ckeditor5-undo",
    "@ckeditor/ckeditor5-utils": "ckeditor/ckeditor5-utils#t/116"
  }
}

And then run mgit update

@pomek pomek mentioned this issue Feb 13, 2017
@pomek pomek closed this as completed in #44 Feb 13, 2017
pomek pushed a commit that referenced this issue Feb 13, 2017
Fix: Fixed various minor issues with the commands. Introduced missing tests. Closes #31. Closes #41. Closes #3. Closes #43.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants