Contribute a Fix and Reviewing changes

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
9 messages Options
Reply | Threaded
Open this post in threaded view
|

Contribute a Fix and Reviewing changes

Nicolai Hess-3-2
Hi,

I tried to create a pull request with IceBerg.
I followed the CONTRIBUTING HowTo and Markus' create TechTalk-Video.

But I must have done something wrong.
This is my pull request:
https://github.com/pharo-project/pharo/pull/1189
But although I just changes one line in one method.
The gitub pull request side shows
Commit 1 Files changed 40!
But I did not change 40 files


And I still have problems viewing pull requests from within pharo:






I thought this would work meanwhil.


Thanks in advance

nicolai

Reply | Threaded
Open this post in threaded view
|

Re: Contribute a Fix and Reviewing changes

Peter Uhnák
Hi,

did you commit on Windows? Because Iceberg seems to commit in platform-specific line endings, so I always need to convert it manually... ( https://github.com/PowerShell/Win32-OpenSSH/wiki/Dos2Unix---Text-file-format-converters )

Notice also that all the files are from the same directory ... when you commit even a single change, the entire package is filed out.

Peter




On Tue, Apr 10, 2018 at 11:16 PM, Nicolai Hess <[hidden email]> wrote:
Hi,

I tried to create a pull request with IceBerg.
I followed the CONTRIBUTING HowTo and Markus' create TechTalk-Video.

But I must have done something wrong.
This is my pull request:
https://github.com/pharo-project/pharo/pull/1189
But although I just changes one line in one method.
The gitub pull request side shows
Commit 1 Files changed 40!
But I did not change 40 files


And I still have problems viewing pull requests from within pharo:






I thought this would work meanwhil.


Thanks in advance

nicolai


Reply | Threaded
Open this post in threaded view
|

Re: Contribute a Fix and Reviewing changes

Nicolai Hess-3-2
Ah!
Ok.

2018-04-10 23:26 GMT+02:00 Peter Uhnák <[hidden email]>:
Hi,

did you commit on Windows? Because Iceberg seems to commit in platform-specific line endings, so I always need to convert it manually... ( https://github.com/PowerShell/Win32-OpenSSH/wiki/Dos2Unix---Text-file-format-converters )

And how can I do this, if my changes are managed by iceberg ?
Do I have to manually change the files in my local branch (right at my git directory) and can I then still upload the fix from within iceberg?
 

Notice also that all the files are from the same directory ... when you commit even a single change, the entire package is filed out.

Peter




On Tue, Apr 10, 2018 at 11:16 PM, Nicolai Hess <[hidden email]> wrote:
Hi,

I tried to create a pull request with IceBerg.
I followed the CONTRIBUTING HowTo and Markus' create TechTalk-Video.

But I must have done something wrong.
This is my pull request:
https://github.com/pharo-project/pharo/pull/1189
But although I just changes one line in one method.
The gitub pull request side shows
Commit 1 Files changed 40!
But I did not change 40 files


And I still have problems viewing pull requests from within pharo:






I thought this would work meanwhil.


Thanks in advance

nicolai



Reply | Threaded
Open this post in threaded view
|

Re: Contribute a Fix and Reviewing changes

EstebanLM
yes, that’s why I asked about the problem with line endings. 

I will solve it, heh… don’t worry. 

In the mean time I will fix the PR tomorrow.

Esteban

On 10 Apr 2018, at 23:31, Nicolai Hess <[hidden email]> wrote:

Ah!
Ok.

2018-04-10 23:26 GMT+02:00 Peter Uhnák <[hidden email]>:
Hi,

did you commit on Windows? Because Iceberg seems to commit in platform-specific line endings, so I always need to convert it manually... ( https://github.com/PowerShell/Win32-OpenSSH/wiki/Dos2Unix---Text-file-format-converters )

And how can I do this, if my changes are managed by iceberg ?
Do I have to manually change the files in my local branch (right at my git directory) and can I then still upload the fix from within iceberg?
 

Notice also that all the files are from the same directory ... when you commit even a single change, the entire package is filed out.

Peter




On Tue, Apr 10, 2018 at 11:16 PM, Nicolai Hess <[hidden email]> wrote:
Hi,

I tried to create a pull request with IceBerg.
I followed the CONTRIBUTING HowTo and Markus' create TechTalk-Video.

But I must have done something wrong.
This is my pull request:
https://github.com/pharo-project/pharo/pull/1189
But although I just changes one line in one method.
The gitub pull request side shows
Commit 1 Files changed 40!
But I did not change 40 files


And I still have problems viewing pull requests from within pharo:



<grafik.png>


I thought this would work meanwhil.


Thanks in advance

nicolai




Reply | Threaded
Open this post in threaded view
|

Re: Contribute a Fix and Reviewing changes

Peter Uhnák
In reply to this post by Nicolai Hess-3-2
Hi,

I --amend(ed) the commit after I converted it. If you've already pushed, you can force-push (there's no one using your code yet, so forcing shouldn't be an issue).
If you have multiple commits, then you'd need to filter all the commits (filter-branch iirc), so the newlines are correct in every single commit.

Peter

On Tue, Apr 10, 2018 at 11:31 PM, Nicolai Hess <[hidden email]> wrote:
Ah!
Ok.

2018-04-10 23:26 GMT+02:00 Peter Uhnák <[hidden email]>:
Hi,

did you commit on Windows? Because Iceberg seems to commit in platform-specific line endings, so I always need to convert it manually... ( https://github.com/PowerShell/Win32-OpenSSH/wiki/Dos2Unix---Text-file-format-converters )

And how can I do this, if my changes are managed by iceberg ?
Do I have to manually change the files in my local branch (right at my git directory) and can I then still upload the fix from within iceberg?
 

Notice also that all the files are from the same directory ... when you commit even a single change, the entire package is filed out.

Peter




On Tue, Apr 10, 2018 at 11:16 PM, Nicolai Hess <[hidden email]> wrote:
Hi,

I tried to create a pull request with IceBerg.
I followed the CONTRIBUTING HowTo and Markus' create TechTalk-Video.

But I must have done something wrong.
This is my pull request:
https://github.com/pharo-project/pharo/pull/1189
But although I just changes one line in one method.
The gitub pull request side shows
Commit 1 Files changed 40!
But I did not change 40 files


And I still have problems viewing pull requests from within pharo:






I thought this would work meanwhil.


Thanks in advance

nicolai




Reply | Threaded
Open this post in threaded view
|

Re: Contribute a Fix and Reviewing changes

Peter Uhnák
Esteban, maybe (until the issue is properly resolved) Iceberg could have a conditional check that the repo is pharo itself and just forced the output to be LF?

Peter

On Tue, Apr 10, 2018 at 11:36 PM, Peter Uhnák <[hidden email]> wrote:
Hi,

I --amend(ed) the commit after I converted it. If you've already pushed, you can force-push (there's no one using your code yet, so forcing shouldn't be an issue).
If you have multiple commits, then you'd need to filter all the commits (filter-branch iirc), so the newlines are correct in every single commit.

Peter

On Tue, Apr 10, 2018 at 11:31 PM, Nicolai Hess <[hidden email]> wrote:
Ah!
Ok.

2018-04-10 23:26 GMT+02:00 Peter Uhnák <[hidden email]>:
Hi,

did you commit on Windows? Because Iceberg seems to commit in platform-specific line endings, so I always need to convert it manually... ( https://github.com/PowerShell/Win32-OpenSSH/wiki/Dos2Unix---Text-file-format-converters )

And how can I do this, if my changes are managed by iceberg ?
Do I have to manually change the files in my local branch (right at my git directory) and can I then still upload the fix from within iceberg?
 

Notice also that all the files are from the same directory ... when you commit even a single change, the entire package is filed out.

Peter




On Tue, Apr 10, 2018 at 11:16 PM, Nicolai Hess <[hidden email]> wrote:
Hi,

I tried to create a pull request with IceBerg.
I followed the CONTRIBUTING HowTo and Markus' create TechTalk-Video.

But I must have done something wrong.
This is my pull request:
https://github.com/pharo-project/pharo/pull/1189
But although I just changes one line in one method.
The gitub pull request side shows
Commit 1 Files changed 40!
But I did not change 40 files


And I still have problems viewing pull requests from within pharo:






I thought this would work meanwhil.


Thanks in advance

nicolai





Reply | Threaded
Open this post in threaded view
|

Re: Contribute a Fix and Reviewing changes

EstebanLM


On 10 Apr 2018, at 23:37, Peter Uhnák <[hidden email]> wrote:

Esteban, maybe (until the issue is properly resolved) Iceberg could have a conditional check that the repo is pharo itself and just forced the output to be LF?

yes, it could. 
I will add that tomorrow *if* libgit does not follow .gitattributes.

Esteban


Peter

On Tue, Apr 10, 2018 at 11:36 PM, Peter Uhnák <[hidden email]> wrote:
Hi,

I --amend(ed) the commit after I converted it. If you've already pushed, you can force-push (there's no one using your code yet, so forcing shouldn't be an issue).
If you have multiple commits, then you'd need to filter all the commits (filter-branch iirc), so the newlines are correct in every single commit.

Peter

On Tue, Apr 10, 2018 at 11:31 PM, Nicolai Hess <[hidden email]> wrote:
Ah!
Ok.

2018-04-10 23:26 GMT+02:00 Peter Uhnák <[hidden email]>:
Hi,

did you commit on Windows? Because Iceberg seems to commit in platform-specific line endings, so I always need to convert it manually... ( https://github.com/PowerShell/Win32-OpenSSH/wiki/Dos2Unix---Text-file-format-converters )

And how can I do this, if my changes are managed by iceberg ?
Do I have to manually change the files in my local branch (right at my git directory) and can I then still upload the fix from within iceberg?
 

Notice also that all the files are from the same directory ... when you commit even a single change, the entire package is filed out.

Peter




On Tue, Apr 10, 2018 at 11:16 PM, Nicolai Hess <[hidden email]> wrote:
Hi,

I tried to create a pull request with IceBerg.
I followed the CONTRIBUTING HowTo and Markus' create TechTalk-Video.

But I must have done something wrong.
This is my pull request:
https://github.com/pharo-project/pharo/pull/1189
But although I just changes one line in one method.
The gitub pull request side shows
Commit 1 Files changed 40!
But I did not change 40 files


And I still have problems viewing pull requests from within pharo:



<grafik.png>


I thought this would work meanwhil.


Thanks in advance

nicolai






Reply | Threaded
Open this post in threaded view
|

Re: Contribute a Fix and Reviewing changes

Stephane Ducasse-3
In reply to this post by Nicolai Hess-3-2
Hi nicolai

Keep pushing :) I succeeded to make it work and now I'm eager to use the new UI 
and refactored version. 

Stef

On Tue, Apr 10, 2018 at 11:16 PM, Nicolai Hess <[hidden email]> wrote:
Hi,

I tried to create a pull request with IceBerg.
I followed the CONTRIBUTING HowTo and Markus' create TechTalk-Video.

But I must have done something wrong.
This is my pull request:
https://github.com/pharo-project/pharo/pull/1189
But although I just changes one line in one method.
The gitub pull request side shows
Commit 1 Files changed 40!
But I did not change 40 files


And I still have problems viewing pull requests from within pharo:






I thought this would work meanwhil.


Thanks in advance

nicolai


Reply | Threaded
Open this post in threaded view
|

Re: Contribute a Fix and Reviewing changes

EstebanLM
tomorrow I will present it to the world ;)
and in theory it will be released, but today the CI was dead :( 

Esteban

On 11 Apr 2018, at 21:29, Stephane Ducasse <[hidden email]> wrote:

Hi nicolai

Keep pushing :) I succeeded to make it work and now I'm eager to use the new UI 
and refactored version. 

Stef

On Tue, Apr 10, 2018 at 11:16 PM, Nicolai Hess <[hidden email]> wrote:
Hi,

I tried to create a pull request with IceBerg.
I followed the CONTRIBUTING HowTo and Markus' create TechTalk-Video.

But I must have done something wrong.
This is my pull request:
https://github.com/pharo-project/pharo/pull/1189
But although I just changes one line in one method.
The gitub pull request side shows
Commit 1 Files changed 40!
But I did not change 40 files


And I still have problems viewing pull requests from within pharo:



<grafik.png>


I thought this would work meanwhil.


Thanks in advance

nicolai