RBTools Ticket #4700: Add a flag (or modify existing flag) to rbt post to override OPEN_BROWSER setting from config files

2018-06-06 Thread Joe Kuang
--
To reply, visit https://hellosplat.com/s/beanbag/tickets/4700/
--

New ticket #4700 by joekuang
For Beanbag, Inc. > RBTools

Status: New
Tags: Priority:Medium, Type:Enhancement


--
Add a flag (or modify existing flag) to rbt post to override OPEN_BROWSER 
setting from config files
==

# What version are you running?
0.7.11

# Describe the enhancement and the motivation for it.
Adding a flag to allow users to explicitly disable the open browser default 
from the .reviewboardrc file under a repository (such as `rbt post 
--no-browser`). The motivation is to offer users the capability of overriding 
configs like OPEN_BROWSER which potentially breaks the user experience of their 
development tooling if they were working with a repository that contains a 
.reviewboardrc file. Perhaps adding overrides can also apply to other rbt 
arguments.


# Please provide any additional information below.


--

-- 
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at https://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Re: Review Board Ticket #4699: Wrong filtering of opcodes for an interdiff

2018-06-06 Thread Christian Hammond
--
To reply, visit https://hellosplat.com/s/beanbag/tickets/4699/
--

New update by alexG
For Beanbag, Inc. > Review Board > Ticket #4699


Reply:

Thanks for the contribution, but as mentioned above, I have a rewrite of 
the algorith mthat does the right thing. It's aimed for the next 3.0.x release 
(had to delay it for other reasons).

-- 
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at https://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Re: Review Board Ticket #4699: Wrong filtering of opcodes for an interdiff

2018-06-06 Thread Alexander Galperin
--
To reply, visit https://hellosplat.com/s/beanbag/tickets/4699/
--

New update by alexG
For Beanbag, Inc. > Review Board > Ticket #4699


Reply:

I posted review request with fix for filtering issue: 
https://reviews.reviewboard.org/r/1/
I commented out call to 'opcodes = self._apply_processors(opcodes) ' , ( so 
the processors.py file can be removed)
After receiving not filtered interdiff chunks, I created diff chunks, and 
compare changed lines in both chunks(diff and interdiff)
The Lines from interdiff, that do not exist in diff chunk, marked as 'equal'
The new code is much shorter and clear, and create correct results in all 
cases

-- 
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at https://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.