In D25425#12230, @valerio.bozzolan wrote:maybe in the future we can have a CSS selector to be applied just to the "phorge/" pathname, in order to avoid mouse selection, and select just the command.
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Feed Advanced Search
Advanced Search
Advanced Search
Aug 30 2023
Aug 30 2023
In D25426#12221, @avivey wrote:See D25420 for a conflicting change...
I like this approach, and prefer it to my simpler attempt from D25426; but have some minor suggestions:
Aug 29 2023
Aug 29 2023
waldyrious removed a reviewer for D25425: Remove mention of Phabricator in the Auth setup check: Blessed Committers.
IMO the added text should be connected to the second sentence of the existing third paragraph, maybe as a separate paragraph. In fact I'd suggest re-grouping the paragraphs differently, to focus on a brief intro, info for users, and info for contributors. Perhaps something like this:
Aug 18 2023
Aug 18 2023
waldyrious closed T15616: PHP 8.1 "strlen(null)" exception when setting up Phorge locally, a subtask of T15064: Make Phorge compatible with PHP 8.1/8.2/8.3, as Resolved.
waldyrious closed T15616: PHP 8.1 "strlen(null)" exception when setting up Phorge locally as Resolved.
Oops, sorry, I was not on master. I was actually testing out D25079. This is already fixed on master, by @valerio.bozzolan.
valerio.bozzolan awarded T15616: PHP 8.1 "strlen(null)" exception when setting up Phorge locally a Cup of Joe token.
Content licensed under Creative Commons Attribution-ShareAlike 4.0 (CC-BY-SA) unless otherwise noted; code licensed under Apache 2.0 or other open source licenses. · CC BY-SA 4.0 · Apache 2.0