Can only speak for myself: yes, insanely so, I have 19 days left to move datacenter.
merge requests for PLi's git
Re: merge requests for PLi's git #1201
Posted 4 July 2018 - 20:57
Currently in use: VU+ Duo 4K (2xFBC S2), VU+ Solo 4K (1xFBC S2), uClan Usytm 4K Ultimate (S2+T2), Octagon SF8008 (S2+T2), Zgemma H9.2H (S2+T2)
Due to my bad health, I will not be very active at times and may be slow to respond. I will not read the forum or PM on a regular basis.
Many answers to your question can be found in our new and improved wiki.
Re: merge requests for PLi's git #1202
Posted 7 July 2018 - 08:09
I can wish you luck only then
https://github.com/O...e-core/pull/499
https://github.com/O...e-core/pull/501
https://github.com/O...e-core/pull/502
https://github.com/O...igma2/pull/1630
Open Vision sources: https://github.com/OpenVisionE2
Re: merge requests for PLi's git #1203
Posted 7 July 2018 - 08:40
@littlesat https://github.com/O...igma2/pull/1631 updated
Open Vision sources: https://github.com/OpenVisionE2
Re: merge requests for PLi's git #1204
Re: merge requests for PLi's git #1205
Posted 7 July 2018 - 13:51
Open Vision sources: https://github.com/OpenVisionE2
Re: merge requests for PLi's git #1206
Posted 7 July 2018 - 18:43
Open Vision sources: https://github.com/OpenVisionE2
Re: merge requests for PLi's git #1207
Re: merge requests for PLi's git #1208
Posted 8 July 2018 - 05:06
Update Systeminfo.py it seems to provoke an error on building!?
What's the error? My build is fine ...
Open Vision sources: https://github.com/OpenVisionE2
Re: merge requests for PLi's git #1209
Posted 8 July 2018 - 05:22
Open Vision sources: https://github.com/OpenVisionE2
Re: merge requests for PLi's git #1210
Re: merge requests for PLi's git #1211
Posted 8 July 2018 - 06:23
That would be great, will you do it?Maybe we need to consider to
Add a Function FileHas or so... that checks if the file has a string inside (and also returns False or None when it does not exist...)
Also there is another check here: https://github.com/O...ch.py#L215~L218
But I wasn't sure how you want to use it.
Edited by Persian Prince, 8 July 2018 - 06:26.
Open Vision sources: https://github.com/OpenVisionE2
Re: merge requests for PLi's git #1212
Posted 8 July 2018 - 07:09
Open Vision sources: https://github.com/OpenVisionE2
Re: merge requests for PLi's git #1213
Posted 8 July 2018 - 08:03
Open Vision sources: https://github.com/OpenVisionE2
Re: merge requests for PLi's git #1214
Posted 8 July 2018 - 08:13
That would be great, will you do it?Maybe we need to consider to
Add a Function FileHas or so... that checks if the file has a string inside (and also returns False or None when it does not exist...)
Also there is another check here: https://github.com/O...ch.py#L215~L218
But I wasn't sure how you want to use it.
We can use "FileHas" for these lines for example:
https://github.com/O...stemInfo.py#L71
https://github.com/O...stemInfo.py#L73
https://github.com/O...stemInfo.py#L78
https://github.com/O...temInfo.py#L103
https://github.com/O...temInfo.py#L104
https://github.com/O...temInfo.py#L105
Open Vision sources: https://github.com/OpenVisionE2
Re: merge requests for PLi's git #1215
Re: merge requests for PLi's git #1216
Posted 8 July 2018 - 08:22
https://github.com/O...igma2/pull/1639 po update
https://github.com/O...igma2/pull/1640
So "FileHas" should be define here: https://github.com/O...es.py#L172~L182 right?
Open Vision sources: https://github.com/OpenVisionE2
Re: merge requests for PLi's git #1217
Posted 8 July 2018 - 09:24
Open Vision sources: https://github.com/OpenVisionE2
Re: merge requests for PLi's git #1218
Posted 8 July 2018 - 09:50
Open Vision sources: https://github.com/OpenVisionE2
Re: merge requests for PLi's git #1219
Posted 8 July 2018 - 10:02
PP and Littlesat: Watch out for unintended white space change. PR#5 introduced all sorts of spaces or tabs at the end of lines that weren't intended to be changed.
Hint: use the white space signalling function of git, it will mark lines with only whitespace or whitespace beyond the end of a line as red so you can correct it before you commit it.
Edited by Erik Slagter, 8 July 2018 - 10:07.
* Wavefrontier T90 with 28E/23E/19E/13E via SCR switches 2 x 2 x 6 user bands
I don't read PM -> if you have something to ask or to report, do it in the forum so others can benefit. I don't take freelance jobs.
Ik lees geen PM -> als je iets te vragen of te melden hebt, doe het op het forum, zodat anderen er ook wat aan hebben.
Re: merge requests for PLi's git #1220
Posted 8 July 2018 - 10:04
Also, as mentioned before, a description of "Update PluginComponent.py" is unacceptable. This time I will leave but next time -> revert. Descriptions containing "update", "tweak", "optimise" or "streamline" (if not specified any further) should raise the red flag.
A guideline for a good commit description is:
- what did the code do?
- what whas undesirable about that?
- how does this commit change that into desirable behaviour
Edited by Erik Slagter, 8 July 2018 - 10:06.
* Wavefrontier T90 with 28E/23E/19E/13E via SCR switches 2 x 2 x 6 user bands
I don't read PM -> if you have something to ask or to report, do it in the forum so others can benefit. I don't take freelance jobs.
Ik lees geen PM -> als je iets te vragen of te melden hebt, doe het op het forum, zodat anderen er ook wat aan hebben.
37 user(s) are reading this topic
0 members, 36 guests, 0 anonymous users
-
Bing (1)