You are now in the family of Trusted Contributors :) Patch welcome.
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Advanced Search
Today
Yesterday
Mon, Apr 22
(From Task description)In diffusion repositories, Description and tags are hard to find or never shown (only show in Manage page).
In T15526#16806, @aklapper wrote:Which exact page(s) is this ticket about?
Thu, Apr 18
Probably something here should be expanded to check if the form contains at least a visible input without read-only
You are now in the family of Trusted Contributors 🎇
Wed, Apr 17
Thanks again avivey. I liked the idea of using applyExternalEffects(). I have not found a way to do it efficiently, without introducing another static method.
(Credit to bekay indeed)
Nice :) Thanks
Bonus point: maybe instead of "#123" we can use "#L123" since both GitLab and GitHub uses that (and that's not a stupid specification). Just because we can.
If we find 10 minutes, let's test this on every provider, as last time it gave us a little surprise on one of them
Tue, Apr 16
Thanks! It worked now! As mentioned in the chat, I'm quite sure that point n.3 and n.4 should be swapped in the README
Mon, Apr 15
Premising I've already seen this syntax in Java but not in PHP, I'm happy that now your IDE says something useful. Maybe a better title is "Improve PHPDoc of id()".
Fri, Apr 12
I think the purpose is to tell the IDE that the function receives a type and returns that exact type.
(Stupid question :D sorry - clear now)
In D25080#16549, @aklapper wrote:I still think that the two Subscriber Rule value should be a list strings should for consistency be Add subscriber rule value should be a list respectively Remove subscriber rule value should be a list.
Where I can document myself about undefined_magic_text? Undetected with grep 🤔
Thu, Apr 11
fix red color for removal (like in PhabricatorProjectTriggerRemoveProjectsRule)
follow code review - thanks
Mon, Apr 8
Ouch I see that receiving the engine was very complicated.
Sun, Apr 7
Fri, Apr 5
A weird but... expandable, fast, easy possibility is:
Thanks. Tested this in a correct situation and also causing a crash putting an unset($nodes[$current_node]) in ../arcanist/src/utils/AbstractDirectedGraph.php to see the nice exception message
...or Pharkdown :D
Thu, Apr 4
I do not understand why it does not work to me.
Go MoreMimeMailParserPeouw
Wed, Apr 3
Ah, also adding a smalll meta "noindex" HTML tag on legacy file.php$123 pages and similar ones, would maybe help a little bit.
Agree with last comment
add minor comment - ready again - tested
introduce a nice applyExternalEffect() to change external things (the orginal file)
All of this should maybe go under "applyExternalEffects()"
In this way, do we skip that column creating a column shift?
Tue, Apr 2
Whops, typo "Seperate" in the title - sorry for not noticing
Congratulations to both bekay and avivey. Minimum change, maximum result.
beautified even more - all logic inside the Transaction itself
I can reproduce this, on an hosted Subversion repository.
If Phorge adds this feature, it is a good opportunity to be nice to thousands of people who use GitLab and GitHub, to unify with their syntax. Visible here:
I've added additional context since interestingly this feature is available since 2017 in GitLab :O wow
- Are we interested in #fff?
Looking at the mentioned question, it seems a NULL pointer arrives as first argument in phutil_encode_log(). Fortunately that function is used by only 3 places:
Mon, Apr 1
Sun, Mar 31
(Whops! wrong summary)
Sat, Mar 30
Fri, Mar 29
Plenty of creative scope for a follow-up Diff - T15766
I agree and I'm quite sure that just parts is missing