-
Notifications
You must be signed in to change notification settings - Fork 165
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
Changes on jQueryMobile Framework #346
Conversation
New Versions from jquery and jquery mobile German lproj Added some bindings
@@ -1,10 +1,10 @@ | |||
package er.jquerymobile.components; | |||
|
|||
import org.apache.log4j.Logger; | |||
import org.apache.log4j.*; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please don't use asterisk imports.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, need to change my eclipse settings. Should i create a pull request with original imports ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just add an additional commit to your pull request. No need to make a new one. That's why it's often preferable to put the commits for a pull request into a separate branch, as the pull request will always reference the head of the branch you chose during your request.
Am 20.12.2012 um 09:02 schrieb Stefan Klein [email protected]:
In Frameworks/Ajax/ERJQueryMobile/Sources/er/jquerymobile/components/ERQMHeaderBar.java:
@@ -1,10 +1,10 @@
package er.jquerymobile.components;-import org.apache.log4j.Logger;
+import org.apache.log4j.*;
Sorry, need to change my eclipse settings. Should i create a pull request with original imports ?�
Reply to this email directly or view it on GitHub.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hallo Johann,
ich melde mich mal auf Deutsch (geht halt doch schneller ;-) ).
Ich habe noch nicht so viel mit git gearbeitet und jetzt hat er
mir das .gitignore in den commit eingebaut.
Wenn ich den commit zurücknehmen will, erhalte ich die Meldung:
error: The following untracked working tree files would be
overwritten by merge:
.gitignore
Please move or remove them before you can merge.
Wie kann ich das lösen?
Viele Grüße
Stefan
Am 20.12.12 00:38, schrieb Johann Werner:
In
Frameworks/Ajax/ERJQueryMobile/Sources/er/jquerymobile/components/ERQMHeaderBar.java:
> @@ -1,10 +1,10 @@
package er.jquerymobile.components;
-import org.apache.log4j.Logger;
+import org.apache.log4j.*;
Please don't use asterisk imports.
—
Reply to this email directly or view
it on GitHub.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hallo Stefan,
anscheinend hast du lokal diese Datei bereits manuell geändert, so dass der commit nicht ohne Weiteres rückgängig gemacht werden kann. Du solltest die Änderungen an dieser Datei daher zuerst entfernen (wenn du z.B. Tower als Git-Client verwendest ist das ein Rechtsklick auf die Datei und "Discard Changes") und es dann nochmals probieren.
Gruß
jw
Am 20.12.2012 um 10:00 schrieb Stefan Klein [email protected]:
In Frameworks/Ajax/ERJQueryMobile/Sources/er/jquerymobile/components/ERQMHeaderBar.java:
@@ -1,10 +1,10 @@
package er.jquerymobile.components;-import org.apache.log4j.Logger;
+import org.apache.log4j.*;Hallo Johann, ich melde mich mal auf Deutsch (geht halt doch schneller ;-) ). Ich habe noch nicht so viel mit git gearbeitet und jetzt hat er mir das .gitignore in den commit eingebaut. Wenn ich den commit zurücknehmen will, erhalte ich die Meldung: error: The following untracked working tree files would be overwritten by merge: .gitignore Please move or remove them before you can merge. Wie kann ich das lösen? Viele Grüße Stefan Mit freundlichen Grüßen Stefan Klein
________________________________________ Büro für Softwaredesign und Entwicklung Dipl. Geogr. Stefan Klein Im Grund 10 68723 Plankstadt Tel. : +49 6202 18568 Fax : +49 6202 26109 EMail: [email protected] WWW : http://www.buero-sde.de Am 20.12.12 00:38, schrieb Johann Werner: In Frameworks/Ajax/ERJQueryMobile/Sources/er/jquerymobile/components/ERQMHeaderBar.java: @@ -1,10 +1,10 @@
package er.jquerymobile.components; -import org.apache.log4j.Logger; +import org.apache.log4j.*;
Please don't use asterisk imports. — Reply to this email directly or view it on GitHub.
—
Reply to this email directly or view it on GitHub.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Johann, thanks for the tip. I tried it already with GitHub and commandline, but the only way I get it in the repository was to commit the gitignore file, change the local file to the original content from wonder and commit again.
Sorry for that, hope you will give the pull request a chance anyway.
The next time i will give Tower a chance.
Stefan
Changes on jQueryMobile Framework
New Versions from jquery and jquery mobile
German lproj
Added some bindings