Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [tycho-dev] gerrit @ eclipse

I am getting server error if I do that. From the message it appears that
I have to vote +/-1. Not sure if the comment gets published or not.

--
Regards,
Igor

On 12-04-17 7:59 AM, Sievers, Jan wrote:
yes. when you hit "Review", you can add a "Cover Message"

Jan

-----Original Message-----
From: tycho-dev-bounces@xxxxxxxxxxx [mailto:tycho-dev-bounces@xxxxxxxxxxx] On Behalf Of Igor Fedorenko
Sent: Dienstag, 17. April 2012 12:26
To: tycho-dev@xxxxxxxxxxx
Subject: Re: [tycho-dev] gerrit @ eclipse

Is there a way to comment on entire proposed change, i.e. not a specific
line?

--
Regards,
Igor

On 12-04-13 8:02 AM, Sievers, Jan wrote:
go back to the patchset overview and hit "Review", then "Publish Comments"

Jan

-----Original Message-----
From: tycho-dev-bounces@xxxxxxxxxxx [mailto:tycho-dev-bounces@xxxxxxxxxxx] On Behalf Of Igor Fedorenko
Sent: Freitag, 13. April 2012 13:40
To: tycho-dev@xxxxxxxxxxx
Subject: Re: [tycho-dev] gerrit @ eclipse

I must be blind, but I still don't see a way to publish comments, only
edit and discard them.

This is probable what I _should_ be doing with the most of the comments
I make... maybe Gerrit just knows better. :-)

--
Regards,
Igor

On 12-04-13 7:31 AM, Sievers, Jan wrote:
just double-click on the line you want to comment

Jan

-----Original Message-----
From: tycho-dev-bounces@xxxxxxxxxxx [mailto:tycho-dev-bounces@xxxxxxxxxxx] On Behalf Of Igor Fedorenko
Sent: Freitag, 13. April 2012 13:18
To: tycho-dev@xxxxxxxxxxx
Subject: Re: [tycho-dev] gerrit @ eclipse

Dumb question. How do I add comments to changes under review? I can't
find a way to do this in Gerrit web ui. Can somebody post a screenshot
of what I should be looking for?

--
Regards,
Igor

On 12-03-09 4:34 AM, Oberlies, Tobias wrote:
Configuration chages requrested: https://bugs.eclipse.org/bugs/show_bug.cgi?id=373748

-----Original Message-----
From: tycho-dev-bounces@xxxxxxxxxxx [mailto:tycho-dev-bounces@xxxxxxxxxxx]
On Behalf Of Mykola Nikishov
Sent: Donnerstag, 8. März 2012 20:53
To: tycho-dev@xxxxxxxxxxx
Subject: Re: [tycho-dev] gerrit @ eclipse

On 03/08/2012 02:47 PM, Oberlies, Tobias wrote:

Do you really mean 'to provide a Change-Id _manually_'? Just install the
commit-msg hook [1] and you're done.

I know that there are ways... Unfortunately, egit is not an option on Windows because we only want LF in the Tycho git repositories and egit still doesn't support core.autocrlf [2]

Regards
Tobias

[2] https://bugs.eclipse.org/bugs/show_bug.cgi?id=301775

[1]
http://wiki.eclipse.org/Gerrit#Install_the_commit-
msg_hook_in_your_repository

--
Mykola

http://ua.linkedin.com/in/manandbytes
https://github.com/manandbytes/
_______________________________________________
tycho-dev mailing list
tycho-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/tycho-dev
_______________________________________________
tycho-dev mailing list
tycho-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/tycho-dev
_______________________________________________
tycho-dev mailing list
tycho-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/tycho-dev
_______________________________________________
tycho-dev mailing list
tycho-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/tycho-dev
_______________________________________________
tycho-dev mailing list
tycho-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/tycho-dev
_______________________________________________
tycho-dev mailing list
tycho-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/tycho-dev
_______________________________________________
tycho-dev mailing list
tycho-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/tycho-dev


Back to the top