text:
DOC-4879,DOC-2933 update metrics for v22.1 and v21.2

Addresses: DOC-4879, DOC-2933

Added 715 v22.1 metrics found in chartcatalog HTTP endpoint but not listed in _includes/v22.1/metric-names.md, bringing the total v22.1 metrics listed to 978.
Added 558 v21.2 metrics found in chartcatalog HTTP endpoint but not listed in _includes/v21.2/metric-names.md, bringing the total v21.2 metrics listed to 818.

Metrics are listed in their proper alpha sort location, and include their help description in full.
QUESTIONS:

Do we really want to publish all 978 || 818 metrics? I initially thought about limiting by specific pkg subdirectory, but was told to consume chartcatalog instead. Well, here it is.

chartcatalog doesn't appear to  exist in v21.1. So this PR does not attempt to address that version. What should be done for v21.1 metrics?

Layout is cumbersome for verylong metric names and also verylong metrics descriptions. Clearly the present form will not suffice, but I am not sure where next to go with this.

This change is

authors: andf-crl cockroach-teamcity


text:
feature request: JSON Schema binding/generation/validation

This maybe should be a plug-in, but I am filing anyway in case others follow down this path...
I'm using JSON Schema for data validation, and also Pane generation.
For example, I wrote a function which generates a new Pane based on a JSON Schema definition.
The generated Pane is a form for creating a new instance of the schema object. Then I validate the resulting data (pulled manually until #431 is fixed) against the schema.
If I pass data along with the schema, the Pane will be filled with data for editing by the user.
This way I can rapidly build fairly robust interfaces with Tweakpane which conform to external data definitions.
Having schema support built-in somehow may be worth considering.

Right. It's not primitive and it should be handled by the plugin system as you said.
BTW I use issues mainly to handle bugs. I'd like to keep the issue list clean but I can't close feature requests. I think GitHub Discussions is the better place to discuss features.

source: gharchive/issue

authors: autonome cocopon

repo: cocopon/tweakpane

license: mit

license_type: permissive

license_source: bigquery


text:
Add Java goody that allows generating an integer's digits

I'd probably suggest using IntStream.iterate for this.

Sure, thank you!

license: MIT

license_source: github-api


text:
Binary for newbies

Well, it could have been nice if there was compiled program of code-cracker, for newbies like me.
I've tried what I could, used VIsual Studio 2017 to compile these repository, but it fails, throws some dependency errors and etc...
why there is not simple a link to download latest compiled version, like all other programs on github, include a link to download the compiled version for people like me, that saves hours for us.
thanks.

To use it, just add the nuget package to your project. See the steps for installing on the main page.

Also, every release has a compiled version:
https://github.com/code-cracker/code-cracker/releases
And also, every build produces artifacts:
https://ci.appveyor.com/project/code-cracker/code-cracker/build/artifacts

authors: giggio jwooley tazotodua

repo: code-cracker/code-cracker


text:
Has anyone recently been able to get Invoker to work? Or is this defunct?

I'm trying to migrate from POW, but lets give Invoker the benefit of the doubt and greenfield a new app rails create newapp && cd newapp. I did..
gem install invoker

My invoker.ini says..
[whisper]
directory = /Users/mycomputer/Sites/newapp
command = bundle exec rails s

I run sudo invoker setup
I run the app with .. invoker start invoker.ini
Looks like it's booting up!
whisper : => Booting Thin
whisper : => Rails 4.0.13 application starting in development on http://0.0.0.0:9000
whisper : => Run `rails server -h` for more startup options
whisper : => Ctrl-C to shutdown server
whisper : Thin web server (v1.6.3 codename Protein Powder)
whisper : Maximum connections set to 1024
whisper : Listening on 0.0.0.0:9000, CTRL+C to stop

But... curl whisper.dev -> Operation Timeout.
Strange. https://0.0.0.0:9000 -> This site can’t provide a secure connection 0.0.0.0 sent an invalid response.
Weird. http://0.0.0.0:9000/ -> Boots up my homepage without SSL support. So anything CORS fails. Once I try and click on a subdomain. It's a big fail.

You can't retrieve https://0.0.0.0:9000 because obviously thin doesn't do https. So that part is correct.
However whisper.dev should work. Which OS is this? Since you said Pow, I am assuming this is OSX. Also which version of OSX?
@iffyuva @kgrz can you double check the behaviour? I don't use OSX anymore..

OSX 10.12.5 (16F73) - Sierra
Good point about Thin. I didn't think of that.

I updated my command as so
[whisper]
directory = /Users/peanut_butter/Sites/whisper.me
command = bash -lc "rvm ruby-2.1.5@whisper do bundle exec thin start --ssl"

Seems to try and boot an SSL version, but alas, no signed SSL so Chrome displays its warning page, and I can only access it by going directly to 0.0.0.0:3000 . No subdomains work. No whisper.dev.

@gotoAndBliss The CORS errors are expected when you request AJAX resources from a different domain (in this case, whisper.dev), depending on the server IP and port. Say if you have your API server proxied to whisper.dev/api, you might not get any CORS errors.
I'm trying to understand if this will be a problem of invoker or not. Up till this point, it doesn't seem so.

@kgrz Ha, I'm six hours deep into this. I would do anything to even have it boot under whisper.dev . It seems to only work as http://0.0.0.0:9000, and then the rest of the site is inaccessible because it's all subdomains that need a whisper.dev domain.

Hmm, so two problems here. You can't access the page on whisper.dev. This either could mean that the page shows a "invoker can't find a process" error, or that it's making a request to your rails server, but all the JS AJAX calls are failing for some reason. What among these two best describes the issue you're facing?

Also, when you mentioned this:

Seems to try and boot an SSL version, but alas, no signed SSL so Chrome displays its warning page

Did you click on the "ADVANCED" link on that warning, and hit "proceed"? I'm trying to map the exact issue you're facing.

Thank you @kgrz for your attention.
I'm using this as my config now
[whisper]
directory = /Users/peanut_butter/Sites/whisper.me
command = bash -lc "rvm ruby-2.1.5@whisper do bundle exec thin start --ssl"

Which boots up as so
Starting process - whisper waiting for 1 seconds...
whisper : Warning! PATH is not properly set up, '/Users/peanut_butter/.rvm/gems/ruby-2.1.5@whisper/bin' is not at first place.
whisper :          Usually this is caused by shell initialization files. Search for 'PATH=...' entries.
whisper :          You can also re-add RVM to your profile by running: 'rvm get stable --auto-dotfiles'.
whisper :          To fix it temporarily in this shell session run: 'rvm use ruby-2.1.5@whisper'.
whisper :          To ignore this error add rvm_silence_path_mismatch_check_flag=1 to your ~/.rvmrc file.
whisper : Using rack adapter
whisper : DEPRECATION WARNING: Support for Rails < 4.1.0 will be dropped. (called from <top (required)> at /Users/peanut_butter/Sites/whisper.me/config/application.rb:5)
whisper : Thin web server (v1.6.3 codename Protein Powder)
whisper : Maximum connections set to 1024
whisper : Listening on 0.0.0.0:3000, CTRL+C to stop

If I go to whisper.dev or https://whisper.dev , I see in Chrome..
This site can’t be reached
This site on the company, organization or school intranet has the same URL as an external website.
Try contacting your system administrator.

This either could mean that the page shows a "invoker can't find a process" error, or that it's making a request to your rails server, but all the JS AJAX calls are failing for some reason. What among these two best describes the issue you're facing?

I do not seem to be getting any invoker errors. The Invoker server does not get hit when I go to whisper.dev .
If I run scutil --dns
I see no .dev extensions so that tells me that maybe Invoker is not creating the DNS at all.

Did you click on the "ADVANCED" link on that warning, and hit "proceed"? I'm trying to map the exact issue you're facing.

Yes, eventually I went through which brings me to a broken SSL homepage. Again, this only works if I go to https://0.0.0.0:9000 , and from there my site is inaccessible because I work on subdomains within it. For example subdomain.whisper.dev. Since the TLD doesnt' register, the subdomains won't either.

@gotoAndBliss not having .dev resolver explains it. Im successfully using invoker with https and subdomain support. I need following info:
Do you have these files present? If so, what are their contents?

/etc/resolver/dev
~/.invoker/config
/Library/LaunchDaemons/com.codemancers.invoker.firewall.plist

@gotoAndBliss simply put, we need to validate the setup according to this file https://github.com/code-mancers/invoker/blob/master/lib/invoker/power/setup/osx_setup.rb Also, please turn off and turn on your wifi once. I wen't M$ Windowz way, and restarted my machine once just to be sure :D

@iffyuva
After reinstalling, rebooting a few times, I was able to get dev back. Equally, I am able to get to an Invoker page that states
Application not running
Invoker did not get any response. Please check if the application is running.

And I can get that error page by accessing whisper.dev. Awesome!
/etc/resolver/dev
nameserver 127.0.0.1
port 23400

 cat ~/.invoker/config
---
:http_port: 23403
:https_port: 23404
:tld: dev
:dns_port: 23400

<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
<dict>
<key>Label</key>
<string>com.codemancers.invoker</string>
<key>ProgramArguments</key>
<array>
<string>sh</string>
<string>-c</string>
<string>echo "rdr pass on lo0 inet proto tcp from any to any port 80 -> 127.0.0.1 port 23403
rdr pass on lo0 inet proto tcp from any to any port 443 -> 127.0.0.1 port 23404" | pfctl -a 'com.apple/250.InvokerFirewall' -f - -E</string>
</array>
<key>RunAtLoad</key>
<true/>
<key>UserName</key>
<string>root</string>
</dict>
</plist>

I went ahead and rebooted, and restarted WIFI as well. Same results.
Thanks @iffyuva

Ah, so now the invoker error page is showing up, but requests are not going through? One thing I can point out is that since the port number (9000) is non-80, the invoker.ini should have a port config that is set to 9000.
Before you do that, try accessing  https://whisper.dev:9000

In addition to above check, please paste info of invoker list

@kgrz What do you mean by "non-80"? When I booted up thin on SSL it defaulted to 9000. So I started setting the port to that in the invoker.ini.
To keep things simple, but still produce the same current error ( going to braidio.dev produces the Application Not Running error ).. I removed the port call from invoker.ini so I have this now
[braidio]
directory = /Users/peanut_butter/Sites/whisper.me
command = bash -lc "rvm ruby-2.1.5@whisper do bundle exec thin start --ssl"

Invoker.Ini ->
  +---------------------------------------+-------+------+--------------+---------------------------------------------+
  | dir                                   | pid   | port | process_name | shell_command                               |
  +---------------------------------------+-------+------+--------------+---------------------------------------------+
  | /Users/peanut_butter/Sites/whisper.me | 14267 |      | braidio      | bash -lc "rvm ruby-2.1.5@whisper do bundl.. |
  +---------------------------------------+-------+------+--------------+---------------------------------------------+

@gotoAndBliss command is wrong! It should have that -p $PORT stuff. port cannot be empty in invoker list. If its empty, invoker doesn't know where to route the request.

@iffyuva Ah ok so I added the port like so
[braidio]
directory = /Users/peanut_butter/Sites/whisper.me
command = bash -lc "rvm ruby-2.1.5@whisper do bundle exec thin start --ssl -p $PORT"

Then the 9000 gets added back
 +---------------------------------------+-------+------+--------------+---------------------------------------------+
  | /Users/peanut_butter/Sites/whisper.me | 14998 | 9000 | braidio      | bash -lc "rvm ruby-2.1.5@whisper do bundl.. |
  +---------------------------------------+-------+------+--------------+---------------------------------------------+

And the same error persists

So, does curl whisper.dev work? Also, can you paste the error again? Probably screenshot.

Ah! A major breakthrough.
So curl braidio.dev:9000 gets an empty reply from server. But if I use Chrome, it loads the site.
The only caveat at this point is that the SSL is not configured. But I'm guessing this is something I have to do myself via self-signing and Invoker doesn't do that part out-of-box?

But I'm guessing this is something I have to do myself via self-signing and Invoker doesn't do that part out-of-box?

Yes :)

THANK YOU EVERYONE!!

You don't have to use :9000, invoker takes care of forwarding request from braidio.dev to port 9000.

@iffyuva  That doesn't seem to be true. Application Not Found occurs unless I specify a port.

You can either specify the command like bundle exec rails -p $PORT or you can set the port configuration in invoker.ini

[whisper]
command=your command here
directory=same directory you have
port = 9000

@kgrz Yes I tried both those separately and together, but I must specify the port in the url or it will not find the application.

Hmm, that looks odd. The whole point of the project is to avoid users specifying a port in the URL :(

You don't need to specify ssl params with thin. If you want to use https then invoker can take care of that. The ssl connections will be terminated at Invoker and forwarded to thin.  i.e - drop the ssl params from thin and try.

So what I am saying is, removing --ssl from thin arguments and then if everything configured properly you should be able to access your website on https://braidio.dev

@gnufied Thanks for the advise. I dropped all the additional SSL info, and it still remains an insecure untrusted connection.
Perhaps my browser is caching this sort of certificate? Is there a way to refresh it.
Also, there's no way that just https://braidio.dev works. The port always has to be included. :(

Perhaps Invoker doesn't support thin?

Invoker doesn't care about web servers in general - so yeah it does support Thin.
I suspect, what is happening is - there is some left over pf filter rules by Pow and Invoker isn't the one that is receiving the request when you access your service using https://braidio.dev. So there is another filter rule which is directing traffic on port 80 and port 443 to elsewhere.  Make sure you uninstalled pow properly and restarted the network.

I think https://github.com/code-mancers/invoker/issues/162 bug has more details about this problem.

@gnufied I just combed through the pow uninstall script, and then double checked my pf.conf and anywhere POW may place anchors can confirm everything is back to its original vanilla state. I ran through the script manually, then rebooted to double check. Same situation.
Would you know where else a filter rule might be hiding or how one could test for that via terminal?

Port forwarding has suddenly stopped working for me too.  Bummer.

http://myapp.dev:9000/ works
http://myapp.dev/ doesn't work

Resolver and firewall.plist are both in sync with ~/.invoker/config.

@swanandp since which version are you facing this issue?

It just stopped working for me today, I upgraded to latest version, but w/o a fix.

@swanandp let me try to reproduce the issue, and update you. Thanks.

Let me know if you want any debugging information. Might help others

@swanandp sure. For now, please rollback to old version, while we debug this issue.

@iffyuva How does one rollback? And what version would be last working?

I have invoker 1.5.1. Can you try that version @gotoAndBliss
You follow these steps
$ gem install invoker -v 1.5.1

$ invoker -v
1.5.3
$ invoker _1.5.1_ -v
1.5.1
$ invoker _1.5.1_ start <ini file>

@iffyuva Ah thanks so much. Tried 1.5.1, and 1.4.1, still need to assign a port though.

@gotoAndBliss @swanandp I have pushed a branch for debugging em-proxy gem. Can you send me logs for your app(s). Steps:
> git clone git@github.com:code-mancers/invoker.git
> cd invoker
> bundle
> DEBUG_EM_PROXY=1 ./bin/invoker start <your-ini-file-path-here>

You should see sample logs like this: https://gist.github.com/iffyuva/c6327944a998c83411e1b4d4f045304e
Please gist them, and post them here. Thanks.

@iffyuva I am pretty sure this is a networking setup issue, and not Invoker.  I just uninstalled invoker and tried Pow, and that isn't working as well.  ( This is from my co-working space ).
I will try from home setup, and if everything works, let you know.

@iffyuva Here you are sir. Thank you so much!!
https://gist.github.com/gotoAndBliss/299c3c95d6008baac88c755fd79b3ad0

So, I got this working after running this:
sudo pfctl -f /etc/pf.conf
sudo pfctl -e

Contents of my pf.conf
scrub-anchor "com.apple/*"
nat-anchor "com.apple/*"
rdr-anchor "com.apple/*"
dummynet-anchor "com.apple/*"
anchor "com.apple/*"
load anchor "com.apple" from "/etc/pf.anchors/com.apple"

And before that, I tried enabling port-forwarding: sudo sysctl -w net.inet.ip.fw.enable=1

@swanandp nice that you have solved it yourself :D

@swanandp
Interesting response. I tried sudo sysctl -w net.inet.ip.fw.enable=1, but got sysctl: unknown oid 'net.inet.ip.fw.enable'. I'm using Mac OS X Sierra 10.12.5.
So I started looking for any kind of port forwarding ( I admittedly have no idea what I'm doing ). But I found this.. /etc/sysctl.conf
And messed with this net.link.ether.inet.arp_unicast_lim=0, set it to 1 and ran sudo pfctl -f /etc/pf.conf; sudo pfctl -e but that didn't work.
Out of curiosity, would using RVM somehow affect this? For RVM is pow, they recommend making a .powrc file that writes this ..
if [ -f "$rvm_path/scripts/rvm" ]; then
  source "$rvm_path/scripts/rvm"
  rvm use .
fi

@gotoAndBliss

. Interesting response. I tried sudo sysctl -w net.inet.ip.fw.enable=1, but got sysctl: unknown oid 'net.inet.ip.fw.enable'. I'm using Mac OS X Sierra 10.12.5.

So, oid work in groups as well, you can successively try:
sysctl net
sysctl net.inet
sysctl net.inet.ip

Be warned, there is a lot of output to sift through:
➜  brainy git:(master) sysctl net | wc -l
     431
➜  brainy git:(master) sysctl net.inet | wc -l
     205
➜  brainy git:(master) sysctl net.inet.ip | wc -l
      48
➜  brainy git:(master) sysctl net.inet.ip.forwarding | wc -l
       1
➜  brainy git:(master)

Out of curiosity, would using RVM somehow affect this?

I don't think so.  RVM comes into play after the port forwarding has happened.  It's further up the stack.

@swanandp
I tried this
sudo sysctl -w net.inet.ip.forwarding=1
net.inet.ip.forwarding: 0 -> 1

then
sudo pfctl -f /etc/pf.conf; sudo pfctl -e

But alas, still need to designate the port. My other net.inet.ip 's
https://gist.github.com/gotoAndBliss/049fa7e5ac94b29f0976042f954f019c

Today I went a bit further. Curious if anyone can see something blatantly incorrect about my approach here..
Started also forwarding ip6
sudo sysctl -w net.inet6.ip6.forwarding=1
net.inet6.ip6.forwarding: 0 -> 1

Created a new forwarding rule
sudo touch /private/etc/pf.anchors/org.user.forwarding

With
rdr pass on lo0 inet proto tcp from any to any port 80 -> 127.0.0.1 port 9000
rdr pass on en0 inet proto tcp from any to any port 80 -> 127.0.0.1 port 9000
rdr pass on en1 inet proto tcp from any to any port 80 -> 127.0.0.1 port 9000

Added it to /private/etc/pf.conf
Parse tested it sudo pfctl -vnf /etc/pf.anchors/org.user.forwarding . It was correct.
Rebooted, and ran sudo pfctl -f /etc/pf.conf; sudo pfctl -e .
Still need to include the port 9000 to access domain.

Could it be caused by https://github.com/code-mancers/invoker/issues/184?

@WojtekKruszewski Thanks so much for the recommendation. I was previously on 2.1.5, and went to 2.3.0, but the problem still persisted.

I have bumped into the same issue. I can access my app fine by visiting localhost:9000, but if I visit web.dev I'm greeted by the usual

Application not running
Invoker did not get any response. Please check if the application is running.

I'm using the latest version (1.5.4) and these are my configuration files:
$ cat /etc/resolver/dev
nameserver 127.0.0.1
port 23400

$ cat ~/.invoker/config
---
:http_port: 23403
:https_port: 23404
:tld: dev
:dns_port: 23400

$ cat /Library/LaunchDaemons/com.codemancers.invoker.firewall.plist
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
<dict>
<key>Label</key>
<string>com.codemancers.invoker</string>
<key>ProgramArguments</key>
<array>
<string>sh</string>
<string>-c</string>
<string>echo "rdr pass on lo0 inet proto tcp from any to any port 80 -> 127.0.0.1 port 23403
rdr pass on lo0 inet proto tcp from any to any port 443 -> 127.0.0.1 port 23404" | pfctl -a 'com.apple/250.InvokerFirewall' -f - -E</string>
</array>
<key>RunAtLoad</key>
<true/>
<key>UserName</key>
<string>root</string>
</dict>
</plist>

$ head -1 Procfile
web: sleep 7 && zeus server -p $PORT

and here is the output of invoker list
$ invoker list
  +-----+-----+------+------------------+---------------------------------------------+
  | dir | pid | port | process_name     | shell_command                               |
  +-----+-----+------+------------------+---------------------------------------------+
  |     | 828 | 9000 | web              | sleep 7 && zeus server -p 9000              |
  +-----+-----+------+------------------+---------------------------------------------+

@Arkham Does this work for you: https://github.com/code-mancers/invoker/issues/194#issuecomment-316599726  ( It did for me, but didn't for @gotoAndBliss )

Hey @swanandp! I've tried every suggestion in the thread and the current situation is:

localhost:9000 ✅
web.dev 🔴
web.dev:9000 🔴

@Arkham Are you on a mac or pc?

@Arkham I had the same and was resolved by running with binding 0.0.0.0 (e.g. rails s -b 0.0.0.0)

@Dan2552 amazing, works for me too!

Would there be something equivalent for running rails thin ?

Would there be something similar for running rails thin?

thin also has a -a  option that you can use to specifying binding address.

@gnufied Ah ok, because I tried -a 0.0.0.0, but nothing happened. Should the address always be 0.0.0.0?

Purchased a brand new macbook, fresh install. Same issue. 😭

I discovered that if I greenfielded a new app, everything works as expected. No port required.
I tried stripping down the problem app to practically nothing ( still in the process of stripping ). And everytime I make the call I get this error
whisper : 4736210368:error:1408F09C:SSL routines:ssl3_get_record:http request:ssl/record/ssl3_record.c:322:

Not sure if that means anything to anyone on this thread.

authors: Arkham Braidio Dan2552 WojtekKruszewski gnufied gotoAndBliss iffyuva kgrz swanandp

repo: code-mancers/invoker


text:
Initial crc start stops at "Starting network time synchronization in CodeReady Containers VM"

General information

OS: Linux / Fedora 33 Workstation
Hypervisor: KVM
Did you run crc setup before starting it (Yes)?
Running CRC on: Laptop /  VM (issues occurs in FC33 directly on laptop hardware as well as in KVM virtual machine)

CRC version
CodeReady Containers version: 1.20.0+ef3f80d
OpenShift version: 4.6.6 (embedded in executable)

CRC status
[sander@ysl ~]$ crc status --log-level debug
DEBU CodeReady Containers version: 1.20.0+ef3f80d
DEBU OpenShift version: 4.6.6 (embedded in executable)
Checking file: /home/sander/.crc/machines/crc/.crc-exist
DEBU About to run SSH command:
df -B1 --output=size,used,target /sysroot | tail -1
DEBU SSH command results: err: <nil>, output: 32720793600 14361313280 /sysroot
DEBU About to run SSH command with hidden output
DEBU SSH command failed
DEBU cannot get OpenShift status: ssh command error:
command : oc get co -ojson --context admin --cluster crc --kubeconfig /opt/kubeconfig
err     : Process exited with status 1
output  :
CRC VM:          Running
OpenShift:       Not Reachable (v4.6.6)
Disk Usage:      14.36GB of 32.72GB (Inside the CRC VM)
Cache Usage:     13.69GB
Cache Directory: /home/sander/.crc/cache

CRC config
[sander@ysl ~]$ crc config view
[sander@ysl ~]$ ```
### There is no config view output

## Host Operating System
```bash
[sander@ysl ~]$ cat /etc/os-release
NAME=Fedora
VERSION="33 (Workstation Edition)"
ID=fedora
VERSION_ID=33
VERSION_CODENAME=""
PLATFORM_ID="platform:f33"
PRETTY_NAME="Fedora 33 (Workstation Edition)"
ANSI_COLOR="0;38;2;60;110;180"
LOGO=fedora-logo-icon
CPE_NAME="cpe:/o:fedoraproject:fedora:33"
HOME_URL="https://fedoraproject.org/"
DOCUMENTATION_URL="https://docs.fedoraproject.org/en-US/fedora/f33/system-administrators-guide/"
SUPPORT_URL="https://fedoraproject.org/wiki/Communicating_and_getting_help"
BUG_REPORT_URL="https://bugzilla.redhat.com/"
REDHAT_BUGZILLA_PRODUCT="Fedora"
REDHAT_BUGZILLA_PRODUCT_VERSION=33
REDHAT_SUPPORT_PRODUCT="Fedora"
REDHAT_SUPPORT_PRODUCT_VERSION=33
PRIVACY_POLICY_URL="https://fedoraproject.org/wiki/Legal:PrivacyPolicy"
VARIANT="Workstation Edition"
VARIANT_ID=workstation

Steps to reproduce

crc stop
crc cleanup
crc setup
crc start -p pull-secret

Expected
crc comes up
Actual
crc gets stalled with "INFO starting network time synchronisation in CodeReady Containers VM"
Logs
Before gather the logs try following if that fix your issue
$ crc delete -f
$ crc cleanup
$ crc setup
$ crc start --log-level debug

##Please consider posting the output of crc start --log-level debug  on http://gist.github.com/ and post the link in the issue.
See here: https://gist.github.com/sandervanvugt/aa9887bf254c5c058ef70db916c04273

@sandervanvugt as per your current log gist I am not seeing any error message about Network Time sync, can you make sure it is full log which also consist the error message or upload a fresh one?
Below snip from the logs which you shared shows that it able ot execute the time sync command without any error.
INFO Starting network time synchronization in CodeReady Containers VM
DEBU About to run SSH command:
sudo timedatectl set-ntp on
DEBU SSH command results: err: <nil>

I repeated the procedure. Next gist is here: https://gist.github.com/sandervanvugt/7bce42718d9a09d1a7ea0708d3059f68
You will find the crc start --log-level debug shows the same result as in my previous gist. I have also included the output of a regular crc start, so that you can see that without the --log-level debug it shows "Starting time synchronisation in CodeReady Containers VM". But, as you've mentioned, while using --log-level debug it seems to be coming beyond this point, and it's getting stalled on the "About to run SSH command: host -R 3 foo.apps-crc.testing"

"About to run SSH command: host -R 3 foo.apps-crc.testing"

That sounds like https://github.com/code-ready/crc/issues/1790, but a different cause?
Can we skip this check and see if this would work?

"About to run SSH command: host -R 3 foo.apps-crc.testing"

That sounds like https://github.com/code-ready/crc/issues/1790, but a different cause?
Can we skip this check and see if this would work?

similar issue for
$ crc version
CodeReady Containers version: 1.21.0+68a4cdd7
OpenShift version: 4.6.9 (embedded in executable)

similar issue for
$ crc version
CodeReady Containers version: 1.21.0+68a4cdd7
OpenShift version: 4.6.9 (embedded in executable)

By "similar issue", you mean you are on a linux system, and crc start hangs at About to run SSH command: host -R 3 foo.apps-crc.testing? If not, can you open a new issue providing the requested information? thanks!

By "similar issue", you mean you are also on a linux system, and crc start hangs at About to run SSH command: host -R 3 foo.apps-crc.testing?

Yes, this was the case.

https://github.com/code-ready/crc/issues/1805#issuecomment-753251617
If you ctrl+c once it's hanged, could you try running:
ssh -i ~/.crc/machines/crc/id_ecdsa -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null  core@192.168.130.11 host -R 3 foo.apps-crc.testing

and
ssh -i ~/.crc/machines/crc/id_ecdsa -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null  core@192.168.130.11  sudo podman ps -a

authors: bczoma cfergeau gbraad praveenkumar sandervanvugt

repo: code-ready/crc


text:
Graphics Folder with an implementation of DDA Line Algo in C

Thank you for your contribution. Please provide the details requested below.
ISSUE NUMBER
This is in reference to the conversation we had on Gitter.
SHORT DESCRIPTION
This is a program used to implement the DDA Line algorithms and translations in order to make a robot. The translations occur to points as the robot is made around the base point
TESTING
You just need to make sure you have the graphics.h library before you compile and run this.
If you already have the library, run the program and the outcome can be seen on the screen

@prateekiiest please do have a look at it

show me the output first

This is the  of the output @prateekiiest

source: gharchive/pull-request

authors: prateekiiest rahulkumaran

repo: codeIIEST/Algorithms


text:
Portfolio website desktop version day 1

The website can be seen on both big and small screens too, as the media query used for the desktop version.

Hello
the desktop version does not show any horizontal scroll bar on my pc, it
checked thoroughly,
only the contact form " Get in touch button " needed to be left aligned
beside it no hover or transition is required as per project requirements
please
On Thu, Jan 26, 2023 at 3:49 PM Temitope Ogunleye @.***>
wrote:

@.**** requested changes on this pull request.
Hi @codeWithHyder https://github.com/codeWithHyder,
Great work on making the changes requested by a previous reviewer 👏
You've done well implementing some of the requested changes, but there are
still some which aren't addressed yet.
Highlights

No linter errors. ✔
Followed Github flow ✔
Professional README file. ✔

Required Changes ♻
Check the comments under the review.
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this
PR. However, I strongly recommend you to take them into account as they can
make your code better. Some of them were simply missed by the previous
reviewer and addressing them will really improve your application.
You can also consider:

Nothing to mention.

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something
is not 100% clear.
Please, remember to tag @topeogunleye https://github.com/topeogunleye
in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the
same Pull Request submitted for the first review, either valid or invalid
unless it is requested otherwise.
As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a
limited number of reviews per project (check the exact number in your
Dashboard). If you think that the code review was not fair, you can request
a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s.
In index.html
https://github.com/codeWithHyder/portfolionew/pull/4#discussion_r1087625044
:

@@ -158,41 +164,51 @@ Profesional Art Printing Data More

About me

Hey there! I can help you build a product, feature or website. Look through some of my work and

               experience! If you like

               what you see and have a project you need coded, don’t hesitate to contact me.

As suggested by the previous reviewer, ON HOVER OF BUTTON
when the project is hovered on with the mouse the description and dark
opacity disappear and a See Project button appears to the bottom of the
image and takes the whole image width 👍🏿
On hover your project should look like the card that has the arrow in
the screenshot below, please achieve this functionality for all cards in
this section 👍🏿

Normal State Hover State
https://user-images.githubusercontent.com/17243325/214768181-8b4a0ad9-cf8b-4f28-a299-f1a7b5782152.png
https://user-images.githubusercontent.com/17243325/214768176-dc3f056b-78bb-4e36-be88-cd940bb53f48.png
In index.html
https://github.com/codeWithHyder/portfolionew/pull/4#discussion_r1087631369
:

@@ -202,12 +218,14 @@ About me

I'm always interested in hearing about new projects, so if you'd like to chat please get in touch.

<input type="text" name="user-name" id="user-name" placeholder="Full name" required maxlength="30">

<input type="email" name="email" id="email" placeholder="Email address" required>

<textarea name="message" id="message" maxlength="500" placeholder="Enter text here" required></textarea>

<button id="get-in-touch" name="get-in-touch">Get in touch</button>

I'm always interested in hearing about new projects, so if you'd like to chat please get in touch.

<input type="text" name="user-name" id="user-name" placeholder="Full name" required maxlength="30">

<input type="email" name="email" id="email" placeholder="Email address" required>

<textarea name="message" id="message" maxlength="500" placeholder="Enter text here" required></textarea>

<button id="get-in-touch" name="get-in-touch">Get in touch</button>

As suggested by the previous reviewer, please ensure your contact
form button is aligned properly.

See below
Your Design [image: image]
https://user-images.githubusercontent.com/52934128/214807774-12ad8925-1543-4519-9fd4-28934e5cb901.png
Figma Design [image: image]
https://user-images.githubusercontent.com/52934128/214808272-b6278d96-467a-44a2-bd42-956407ef379b.png
In index.html
https://github.com/codeWithHyder/portfolionew/pull/4#discussion_r1087645364
:

@@ -158,41 +164,51 @@ Profesional Art Printing Data More

About me

Hey there! I can help you build a product, feature or website. Look through some of my work and

               experience! If you like

               what you see and have a project you need coded, don’t hesitate to contact me.

           <button class="gr">Get My Resume</button>

As suggested by the previous reviewer, please kindly update the
font-family on the languages to match the ones on the design template.

Your Design Figma Design
https://user-images.githubusercontent.com/17243325/214769555-d312c7b9-56ce-4fec-9b4e-6b1531872147.png
https://user-images.githubusercontent.com/17243325/214769557-4526ce6f-9498-4445-ab98-8059bd7550dd.png
In style.css
https://github.com/codeWithHyder/portfolionew/pull/4#discussion_r1087647217
:

@@ -493,3 +498,282 @@ form ul li button:hover {

background-color: #fff5e1;
border: 1px solid #f55800;
}

+/* -----------------------------css for desktop or bigger screens-------- */
@.*** only screen and (min-width: 768px) {

main {

width: 100%;

}

.menu {

display: none;

}

As suggested by the previous reviewer, please fix the horizontal
scroll bar as it doesn't make the user experience good.

[image: image]
https://user-images.githubusercontent.com/52934128/214811621-d8652459-1c2a-4988-830f-37cf56e57ffd.png
—
Reply to this email directly, view it on GitHub
https://github.com/codeWithHyder/portfolionew/pull/4#pullrequestreview-1270736455,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZE2WDH2M5YW6KBEDFPEFRDWUJFR5ANCNFSM6AAAAAAUGWYQZA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Very regards
Ghulam Hyder
email #1   * @.**** @.***>

[image: image.png]
On Thu, Jan 26, 2023 at 3:58 PM hyder ali @.***> wrote:

Hello
the desktop version does not show any horizontal scroll bar on my pc, it
checked thoroughly,
only the contact form " Get in touch button " needed to be left aligned
beside it no hover or transition is required as per project requirements
please
On Thu, Jan 26, 2023 at 3:49 PM Temitope Ogunleye <
@.***> wrote:

@.**** requested changes on this pull request.
Hi @codeWithHyder https://github.com/codeWithHyder,
Great work on making the changes requested by a previous reviewer 👏
You've done well implementing some of the requested changes, but there
are still some which aren't addressed yet.
Highlights

No linter errors. ✔
Followed Github flow ✔
Professional README file. ✔

Required Changes ♻
Check the comments under the review.
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this
PR. However, I strongly recommend you to take them into account as they can
make your code better. Some of them were simply missed by the previous
reviewer and addressing them will really improve your application.
You can also consider:

Nothing to mention.

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if
something is not 100% clear.
Please, remember to tag @topeogunleye https://github.com/topeogunleye
in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use
the same Pull Request submitted for the first review, either valid or
invalid unless it is requested otherwise.
As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a
limited number of reviews per project (check the exact number in your
Dashboard). If you think that the code review was not fair, you can request
a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s.
In index.html
https://github.com/codeWithHyder/portfolionew/pull/4#discussion_r1087625044
:

@@ -158,41 +164,51 @@ Profesional Art Printing Data More

About me

Hey there! I can help you build a product, feature or website. Look through some of my work and

               experience! If you like

               what you see and have a project you need coded, don’t hesitate to contact me.

As suggested by the previous reviewer, ON HOVER OF BUTTON
when the project is hovered on with the mouse the description and
dark opacity disappear and a See Project button appears to the bottom of
the image and takes the whole image width 👍🏿
On hover your project should look like the card that has the arrow in
the screenshot below, please achieve this functionality for all cards in
this section 👍🏿

Normal State Hover State
https://user-images.githubusercontent.com/17243325/214768181-8b4a0ad9-cf8b-4f28-a299-f1a7b5782152.png
https://user-images.githubusercontent.com/17243325/214768176-dc3f056b-78bb-4e36-be88-cd940bb53f48.png
In index.html
https://github.com/codeWithHyder/portfolionew/pull/4#discussion_r1087631369
:

@@ -202,12 +218,14 @@ About me

I'm always interested in hearing about new projects, so if you'd like to chat please get in touch.

<input type="text" name="user-name" id="user-name" placeholder="Full name" required maxlength="30">

<input type="email" name="email" id="email" placeholder="Email address" required>

<textarea name="message" id="message" maxlength="500" placeholder="Enter text here" required></textarea>

<button id="get-in-touch" name="get-in-touch">Get in touch</button>

I'm always interested in hearing about new projects, so if you'd like to chat please get in touch.

<input type="text" name="user-name" id="user-name" placeholder="Full name" required maxlength="30">

<input type="email" name="email" id="email" placeholder="Email address" required>

<textarea name="message" id="message" maxlength="500" placeholder="Enter text here" required></textarea>

<button id="get-in-touch" name="get-in-touch">Get in touch</button>

As suggested by the previous reviewer, please ensure your contact
form button is aligned properly.

See below
Your Design [image: image]
https://user-images.githubusercontent.com/52934128/214807774-12ad8925-1543-4519-9fd4-28934e5cb901.png
Figma Design [image: image]
https://user-images.githubusercontent.com/52934128/214808272-b6278d96-467a-44a2-bd42-956407ef379b.png
In index.html
https://github.com/codeWithHyder/portfolionew/pull/4#discussion_r1087645364
:

@@ -158,41 +164,51 @@ Profesional Art Printing Data More

About me

Hey there! I can help you build a product, feature or website. Look through some of my work and

               experience! If you like

               what you see and have a project you need coded, don’t hesitate to contact me.

           <button class="gr">Get My Resume</button>

As suggested by the previous reviewer, please kindly update the
font-family on the languages to match the ones on the design template.

Your Design Figma Design
https://user-images.githubusercontent.com/17243325/214769555-d312c7b9-56ce-4fec-9b4e-6b1531872147.png
https://user-images.githubusercontent.com/17243325/214769557-4526ce6f-9498-4445-ab98-8059bd7550dd.png
In style.css
https://github.com/codeWithHyder/portfolionew/pull/4#discussion_r1087647217
:

@@ -493,3 +498,282 @@ form ul li button:hover {

background-color: #fff5e1;
border: 1px solid #f55800;
}

+/* -----------------------------css for desktop or bigger screens-------- */
@.*** only screen and (min-width: 768px) {

main {

width: 100%;

}

.menu {

display: none;

}

As suggested by the previous reviewer, please fix the horizontal
scroll bar as it doesn't make the user experience good.

[image: image]
https://user-images.githubusercontent.com/52934128/214811621-d8652459-1c2a-4988-830f-37cf56e57ffd.png
—
Reply to this email directly, view it on GitHub
https://github.com/codeWithHyder/portfolionew/pull/4#pullrequestreview-1270736455,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZE2WDH2M5YW6KBEDFPEFRDWUJFR5ANCNFSM6AAAAAAUGWYQZA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Very regards
Ghulam Hyder
email #1   * @.**** @.***>

--
Very regards
Ghulam Hyder
email #1   * @.**** @.***>

Dear @codeWithHyder
I clearly understand the the animations are not the requirements for you to move to the next level. However, as the previous code reviewers suggested. Ensure that when you hover over the card, the texts also disappear. This will make the project look more professional. Refering to the attached video, you can clearly see that the user can get frustrated in case he is not able to read clearly the texts behind the front texts of the card.

Hello the desktop version does not show any horizontal scroll bar on my pc, it checked thoroughly, only the contact form " Get in touch button " needed to be left aligned beside it no hover or transition is required as per project requirements please
…
On Thu, Jan 26, 2023 at 3:49 PM Temitope Ogunleye @.> wrote: @.* requested changes on this pull request. Hi @codeWithHyder https://github.com/codeWithHyder, Great work on making the changes requested by a previous reviewer 👏 You've done well implementing some of the requested changes, but there are still some which aren't addressed yet. Highlights - No linter errors. ✔ - Followed Github flow ✔ - Professional README file. ✔ Required Changes ♻ Check the comments under the review. Optional suggestions Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application. You can also consider: - Nothing to mention. Cheers and Happy coding!👏👏👏 Feel free to leave any questions or comments in the PR thread if something is not 100% clear. Please, remember to tag @topeogunleye https://github.com/topeogunleye in your question so I can receive the notification. Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise. ------------------------------ As described in the Code reviews limits policy https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s. ------------------------------ In index.html <#4 (comment)> : > @@ -158,41 +164,51 @@ Profesional Art Printing Data More    -  - About me - Hey there! I can help you build a product, feature or website. Look through some of my work and - experience! If you like - what you see and have a project you need coded, don’t hesitate to contact me. -  -  - As suggested by the previous reviewer, ON HOVER OF BUTTON when the project is hovered on with the mouse the description and dark opacity disappear and a See Project button appears to the bottom of the image and takes the whole image width 👍🏿 On hover your project should look like the card that has the arrow in the screenshot below, please achieve this functionality for all cards in this section 👍🏿 Normal State Hover State https://user-images.githubusercontent.com/17243325/214768181-8b4a0ad9-cf8b-4f28-a299-f1a7b5782152.png https://user-images.githubusercontent.com/17243325/214768176-dc3f056b-78bb-4e36-be88-cd940bb53f48.png ------------------------------ In index.html <#4 (comment)> : > @@ -202,12 +218,14 @@ About me   -  - I'm always interested in hearing about new projects, so if you'd like to chat please get in touch. -  -  -  - Get in touch +  +  I'm always interested in hearing about new projects, so if you'd like to chat please get in touch. +  +   +  +  + Get in touch - As suggested by the previous reviewer, please ensure your contact form button is aligned properly. See below Your Design [image: image] https://user-images.githubusercontent.com/52934128/214807774-12ad8925-1543-4519-9fd4-28934e5cb901.png Figma Design [image: image] https://user-images.githubusercontent.com/52934128/214808272-b6278d96-467a-44a2-bd42-956407ef379b.png ------------------------------ In index.html <#4 (comment)> : > @@ -158,41 +164,51 @@ Profesional Art Printing Data More    -  - About me - Hey there! I can help you build a product, feature or website. Look through some of my work and - experience! If you like - what you see and have a project you need coded, don’t hesitate to contact me. -  -  - Get My Resume -  - +  - As suggested by the previous reviewer, please kindly update the font-family on the languages to match the ones on the design template. Your Design Figma Design https://user-images.githubusercontent.com/17243325/214769555-d312c7b9-56ce-4fec-9b4e-6b1531872147.png https://user-images.githubusercontent.com/17243325/214769557-4526ce6f-9498-4445-ab98-8059bd7550dd.png ------------------------------ In style.css <#4 (comment)> : > @@ -493,3 +498,282 @@ form ul li button:hover { background-color: #fff5e1; border: 1px solid #f55800; } + +/* -----------------------------css for desktop or bigger screens-------- / @.** only screen and (min-width: 768px) { + main { + width: 100%; + } + + .menu { + display: none; + } + - As suggested by the previous reviewer, please fix the horizontal scroll bar as it doesn't make the user experience good. [image: image] https://user-images.githubusercontent.com/52934128/214811621-d8652459-1c2a-4988-830f-37cf56e57ffd.png — Reply to this email directly, view it on GitHub <#4 (review)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZE2WDH2M5YW6KBEDFPEFRDWUJFR5ANCNFSM6AAAAAAUGWYQZA . You are receiving this because you were mentioned.Message ID: @.>
-- Very regards Ghulam Hyder email #1 * @.* @.***>

Kindly make the other chages pls @codeWithHyder. Maybe it's on my machine that it has an horizontal scrollbar. To avoid this, I suggest that you use percentages for you widths and heights

I can surely do that on my own
But keeping one in review looping frustrates to go a ahead for next
level....
Now I asked for new opinion
On Thu, 26 Jan, 2023, 8:14 pm NTIHINDUKA ALPHA, @.***>
wrote:

Dear @codeWithHyder https://github.com/codeWithHyder
I clearly understand the the animations are not the requirements for you
to move to the next level. However, as the previous code reviewers
suggested. Ensure that when you hover over the card, the texts also
disappear. This will make the project look more professional. Refering to
the attached video, you can clearly see that the user can get frustrated in
case he is not able to read clearly the texts behind the front texts of the
card.
—
Reply to this email directly, view it on GitHub
https://github.com/codeWithHyder/portfolionew/pull/4#issuecomment-1405115001,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZE2WDAL2RJ2SGV3PYPEROLWUKEUHANCNFSM6AAAAAAUGWYQZA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Dear @codeWithHyder
At times code frustrates mostly when trying to fix the bug. Kindly let us know in case you need further support.
Kind regards.

This PR is under a Second Opinion Review.
Kindly refrain from pushing other commits or requesting other reviews until a Second Opinion review is provided.

Thank you for your support!
On Thu, 26 Jan, 2023, 10:25 pm Nicolae Pop, @.***>
wrote:

@.**** commented on this pull request.
Hello @codeWithHyder https://github.com/codeWithHyder 👋,
This is an additional review as a result of the second opinion request.
Kudos for the great effort put into place for this project.~
Kindly note though that the reviewers are right in their demands:

when a user is hovering over the project cards, a "See Project"
button should appear;
the technologies should not have a green border;
there is still an overflow for the desktop version that should not
be there.

https://www.loom.com/share/75ab4a85d90e4ba483b84840ebed5a06
I hope you realize that all these aspects make your portfolio
unprofessional.
But I strongly believe that you will fix all the bugs when you will not
feel all this pressure that you feel now, about advancing to the next
milestone and all.
My opinion is that you can advance to the next milestone though.
As this was the only reason that your project was not approved it will be
approved as soon as possible.
Please note that since we will approve it manually so this could take a
few hours!
Keep rocking 🚀!
—
Reply to this email directly, view it on GitHub
https://github.com/codeWithHyder/portfolionew/pull/4#pullrequestreview-1271429324,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZE2WDENMFMW66FSU5U2WDLWUKT6TANCNFSM6AAAAAAUGWYQZA
.
You are receiving this because you were mentioned.Message ID:
@.***>

authors: AlphaNtihinduka codeWithHyder nicupop729 topeogunleye

repo: codeWithHyder/portfolionew


text:
Ravi Prakash Pal

Description
I have added my profile to this project
Checklist

[x] I have read and followed the Contribution Guidelines.
[x] All new and existing tests passed.
[x] I have updated the documentation to reflect the changes I've made.
[x] My code follows the code style of this project.
[x] The title of my pull request is a short description of the requested changes.

Hi @RaviP9973 , thanks for your interest in contributing to DevDisplay!
Before we proceed, we noticed you adding your profile on #DevDisplay but forgot to star the repo. It's like making an amazing sandwich and forgetting to take a bite! -- I think our repo's feeling a little lonely without your 🤗 So, be sure to star the DevDisplay GitHub Repository. 🌟
💌 This is a mandatory step for all contributors. & It's is the first priority of contribution in DevDisplay 💌
🚀 We look forward to your contribution once these steps are completed. 🚀
Thanks 👍🏻

repo: codeaashu/DevDisplay


text:
Weitere Einstellungen

Zu Veranstaltungen sollen weitere Einstellungen gemacht werden können. Am besten speichern wir diese in einer extra Relation.
Beispiele für Einstellungen

Nahrungsmittelunverträglichkeiten abfragen
weitere Intolleranzen abfragen?
Übernachtungstage abfragen?

#121 könnte das Problem vielleicht lösen


text:
GET request preflight OPTIONS - CORS

Hi guys.
Why do you use preflight option  in simple get request?
Example : https://spa.codecasts.rocks/categories?page=9

Hey @Wit3 !
I assume this is default for the browser. If a preflight is necessary for POST or PUT why not for GET?
If not the browser then Axios.
If you have more details on this matter just add to the issue! Thanks!
Vedovelli

I'm using your project and I'm getting a 301 error on one go my api endpoints I'm using your standard setup with the jet, api and auth middlewares on a Route::resource but it doesn't seem to play well......any suggestions?

authors: Wit3 karneaud vedovelli

repo: codecasts/spa-starter-kit


text:
Avoid using intermediate vars instead object properties in Expressions ( for Where(), Get())

TableQuery.Where and Get methods takes Expression, analyze it and build SQL “where” sentence. Some kind of expressions works on Android but don’t work on iOS. Let’s use classical example with Employee and Department tables (http://www.oracle.com/ocom/groups/public/@otn/documents/digitalasset/513145.jpg). We have 2 corresponded classes with needed attributes. Code:
var dep = connection.Table<Department>().Where(d => d.Name == “blabla”).First();
int count = connection.Table<Employee>().Where(e => e.Department_ID == dep.Department_ID).Count();

This code will work on Android, but wil not on iOS due to problems with AOT compilation in CompileExpr method. To fix it on iOS you can:
var dep = connection.Table().Where(d => d.Name == “blabla”).First();
int depId = dep.Department_ID;
int count = connection.Table().Where(e => e.Department_ID == depId).Count();

It works! And there are question: can we avoid using intermediate var for compability with AOT compilation on iOS?
May be upgrade to latest SQLiteNet version would solve the problem?

Hi @cardinalby, I'm extending the SQLConnection class with some methods that avoid the use of Interfaces so it will be up to you to use whatever you want. Just a question about what you pose here. This solution works even with interface use? Or only without the interfaces?

I can't reproduce the error with my iPad following your first example. It works for me... I'll try to make it crash with a more complicated query. Do you get error with such a query?

I am waiting for IDisposable (commented in https://github.com/codecoding/SQLite4Unity3d/issues/5) fix, then will test on the latest version and prepare the reproducible example.

I can't reproduce it too on last version :). May be it was due to interface usage, which was fixed.

I'm glad but I have to say that I couldn't reproduce it even with the interfaces. In fact I tested it before removing the interfaces just to see if I could learn something. Anyway, I'll close this issue and if you somehow manage to reproduce the behaviour don't hesitate to tell me. I will reopen it again and see how we deal with it.

authors: cardinalby codecoding


text:
Netease batch assign

This component didn't have any tests. I thought hard about how to test and ended up testing manually.
This pull request includes usability improvements.
While processing the emails the user can see progress and the submit button is disabled.
Any emails that cause an error are listed beneath the button after everything has been processed.
This will allow netease to diagnose individual emails.

Closed due to incorrect base branch.

repo: codecombat/codecombat


text:
Update README.md

Update codecov with github actions link

Thanks all!

repo: codecov/example-node


text:
Why not provide rexray/rbd plugin for docker 1.13?

Why not provide rexray/rbd plugin for docker 1.13?

https://hub.docker.com/r/rexray/

Closing as duplicate of #818

authors: JinsYin codenrhoden

repo: codedellemc/rexray


text:
Allow the Scour module to be used as a standalone program

This PR allows to use the Scour module as a "standalone program" (e.g. download, unzip and run) without the need to install the module itself and/or any third party modules.
It consists of two parts:

Use a local copy of "six". We should not expect six to be available in the users Python environment. E.g. Inkscape comes with a minimal installation of Python that does neither include six nor allows the user to easily install it.
Add a simple wrapper script around the Scour module that can be executed directly. The relative imports introduced in d1c66cc75bbe474b5a45a7cd45ed27207311d897 have broken the previous approach to achieve this with if __name__ == '__main__': run()

No, copy-pasting an external dependency (six) into this module is an anti-pattern ..

I don't like it either, but I like the external dependency even less... Do you have a better idea?
Would you accept a minimum set of compatibility code to replace "six" altogether?
Granted, "six" is a small module, but even a small module seems a bit excessive for a module as "simple" as Scour.

As said, duplicating code from a well established external package is a no go.
Rgd the wrapper script: that's already there https://github.com/codedread/scour/blob/master/setup.py#L49

I wasn't suggesting duplicating. I just think that what we currently try to achieve with "six" can be done with much less code in a much easier way. However if you're in favor of an external dependency let's stick to it and see how things play out.
Regarding wrapper script: This script is only created when installing the module, right?
This part of the PR request was specifically for the case where one does not want to install the module (e.g. for development purposes or to run multiple versions of Scour side-by-side on the same system). As a matter of fact I did all my recent development using this wrapper and as #9 indicates this could come in handy for others, too.
Do you have any reason why we shouldn't add it?

This script is only created when installing the module, right?

Yes. It's a feature of distutils.
For development purposes, you can start scour like python -m scour.run.
For multiple versions of scour, there is virtualenv.

Do you have any reason why we shouldn't add it?

Yes: there is an established approach: distutils / console_scripts.

For development purposes, you can start scour like python -m scour.run
I figured that out, too, but only after I searched the Internet on how to do it. It's not an obvious/intuitive solution.

It might well be that your suggested approach is established amongst Python professionals (which you seem to be), but it surely complicates usage of Scour for everybody else.
Personally I always prefer software that keeps it as simple as running a single script/executable and does not need any additional steps like setup / installation of external libraries, etc so it's readily available for everybody (without a deeper understanding of e.g. Python / distutils / console_scripts / pip / whatever)
Maybe we simply differ in opinion here, but maybe it's also something for you to reconsider.

It might well be that your suggested approach is established

Yes, it is.
And "established" == "simple", because of expectations, info on the Web etc etc.
btw: you should be able to directly run the : python scour/scour.py also because of this
https://github.com/codedread/scour/blob/master/scour/scour.py#L3305

btw: you should be able to directly run the : python scour/scour.py also because of this

No, that specific code is actually broken after d1c66cc75bbe474b5a45a7cd45ed27207311d897 as mentioned in my initial description for this PR.
The linked commit introduced relative imports and trying to relatively import from a script does not work (one first has to import the parent module from an absolute path).
As I didn't find an easy way around this I decided on the wrapper script.

Huh? The code is using absolute imports https://github.com/codedread/scour/blob/master/scour/scour.py#L52

@Ede123 I've pushed https://pypi.python.org/pypi/scour/0.31

Huh? The code is using absolute imports

I think https://github.com/codedread/scour/blob/master/scour/scour.py#L59-L60 as well as https://github.com/codedread/scour/blob/master/scour/scour.py#L62 are relative imports, aren't they?
When trying to execute the module as a script those lines error out (with an error message stating that a relative import is not possible without the parent module being imported absolutely; I can give you the exact error message when I get home where I have access to a machine with python installed)

I've pushed https://pypi.python.org/pypi/scour/0.31
Great!

@oberstet: This is the error that is thrown when trying to execute the module as a script:
Traceback (most recent call last):
  File "scour.py", line 59, in <module>
    from .svg_regex import svg_parser
SystemError: Parent module '' not loaded, cannot perform relative import

@Ede123 ah, right. could you retry with trunk pls? just pushed https://github.com/codedread/scour/commit/302f7f7477193165b2b1bdcd5997ee4e1acbf997

I'm afraid this did not do it:
> python scour.py --version
Traceback (most recent call last):
  File "<frozen importlib._bootstrap>", line 2218, in _find_and_load_unlocked
AttributeError: 'module' object has no attribute '__path__'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "scour.py", line 59, in <module>
    from scour.svg_regex import svg_parser
  File "E:\Temp\Downloads\Scour\scour.git\scour\scour.py", line 59, in <module>
    from scour.svg_regex import svg_parser
ImportError: No module named 'scour.svg_regex'; 'scour' is not a package

sure. try: python scour/scour.py --version. anyway: I won't change this. a package is a package.

Exactly the same issue... Did you try that on your machine? If it works I assume it only works because you have the module installed?

works.
oberstet@thinkpad-t430s:~$ ./python279/bin/virtualenv ~/foobar
New python executable in /home/oberstet/foobar/bin/python
Installing setuptools, pip...done.
oberstet@thinkpad-t430s:~$ source ~/foobar/bin/activate
(foobar)oberstet@thinkpad-t430s:~$ which python
/home/oberstet/foobar/bin/python
(foobar)oberstet@thinkpad-t430s:~$ python
Python 2.7.9 (default, Jan 18 2015, 21:40:29)
[GCC 4.8.2] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import scour
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ImportError: No module named scour
>>>
(foobar)oberstet@thinkpad-t430s:~$ which scour
/usr/local/bin/scour
(foobar)oberstet@thinkpad-t430s:~$ cd scm/oberstet/scour/
(foobar)oberstet@thinkpad-t430s:~/scm/oberstet/scour$ git pull
Already up-to-date.
(foobar)oberstet@thinkpad-t430s:~/scm/oberstet/scour$ make test_version
PYTHONPATH=. python -m scour.scour --version
Traceback (most recent call last):
  File "/home/oberstet/python279/lib/python2.7/runpy.py", line 162, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/home/oberstet/python279/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/home/oberstet/scm/oberstet/scour/scour/scour.py", line 60, in <module>
    from scour.svg_transform import svg_transform_parser
  File "scour/svg_transform.py", line 63, in <module>
    from six.moves import range
ImportError: No module named six.moves
make: *** [test_version] Fehler 1
(foobar)oberstet@thinkpad-t430s:~/scm/oberstet/scour$ which pip
/home/oberstet/foobar/bin/pip
(foobar)oberstet@thinkpad-t430s:~/scm/oberstet/scour$ pip install six
You are using pip version 6.0.6, however version 7.1.2 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
Collecting six
  Downloading six-1.10.0-py2.py3-none-any.whl
Installing collected packages: six

Successfully installed six-1.10.0
(foobar)oberstet@thinkpad-t430s:~/scm/oberstet/scour$ make test_version
PYTHONPATH=. python -m scour.scour --version
0.31
(foobar)oberstet@thinkpad-t430s:~/scm/oberstet/scour$ make test_help
PYTHONPATH=. python -m scour.scour --help
scour 0.31
Copyright Jeff Schiller, Louis Simard, 2010
Usage: scour.py [-i input.svg] [-o output.svg] [OPTIONS]

If the input/output files are specified with a svgz extension, then compressed
SVG is assumed. If the input file is not specified, stdin is used. If the
output file is not specified,  stdout is used.

Options:
  --version                   show program's version number and exit
  -h, --help                  show this help message and exit
  --disable-simplify-colors   won't convert all colors to #RRGGBB format
  --disable-style-to-xml      won't convert styles into XML attributes
  --disable-group-collapsing  won't collapse <g> elements
  --create-groups             create <g> elements for runs of elements with
                              identical attributes
  --enable-id-stripping       remove all un-referenced ID attributes
  --enable-comment-stripping  remove all <!-- --> comments
  --shorten-ids               shorten all ID attributes to the least number of
                              letters possible
  --shorten-ids-prefix=SHORTEN_IDS_PREFIX
                              shorten all ID attributes with a custom prefix
  --disable-embed-rasters     won't embed rasters as base64-encoded data
  --keep-unreferenced-defs    won't remove elements within the defs container
                              that are unreferenced
  --keep-editor-data          won't remove Inkscape, Sodipodi or Adobe
                              Illustrator elements and attributes
  --remove-metadata           remove <metadata> elements (which may contain
                              license metadata etc.)
  --renderer-workaround       work around various renderer bugs (currently
                              only librsvg) (default)
  --no-renderer-workaround    do not work around various renderer bugs
                              (currently only librsvg)
  --strip-xml-prolog          won't output the <?xml ?> prolog
  --enable-viewboxing         changes document width/height to 100%/100% and
                              creates viewbox coordinates
  -p DIGITS, --set-precision=DIGITS
                              set number of significant digits (default: 5)
  -q, --quiet                 suppress non-error output
  --indent=INDENT_TYPE        indentation of the output: none, space, tab
                              (default: space)
  --nindent=INDENT_DEPTH      depth of the indentation, i.e. number of
                              spaces/tabs: (default: 1)
  --no-line-breaks            do not create line breaks in output (also
                              disables indentation; might be overriden by
                              xml:space="preserve")
  --strip-xml-space           strip the xml:space="preserve" attribute from
                              the root SVG element
  --protect-ids-noninkscape   Don't change IDs not ending with a digit
  --protect-ids-list=PROTECT_IDS_LIST
                              Don't change IDs given in a comma-separated list
  --protect-ids-prefix=PROTECT_IDS_PREFIX
                              Don't change IDs starting with the given prefix
(foobar)oberstet@thinkpad-t430s:~/scm/oberstet/scour$

works.

Uhm... are you kidding me? You did not even use the command line you provided me in your example. Instead you used python -m scour.scour --version (which will only work if the scour module is on the python search patch).

anyway: I won't change this. a package is a package.

I accept that... that's why I wanted to add a small wrapper script which makes use of the package - In my opinion a reasonable solution that would not contradict your point of view but would allow to easily use the package...

repo: codedread/scour


text:
Switch to Puma for the webserver

Would love it if you gave this a once-over, @daguar. Seems to work, but you know way more about the right ways to do this than I. This is mostly following Heroku’s guide. Left the ActiveRecord stuff in for reference since I believe #31 will involve integrating it.

Pinging @daguar… if you don’t have time to look over this, just let me know. It runs, so I’ll merge.

Like Beetlejuice, you need only repeat my name — it looks all good! I agree with removing the ActiveRecord stuff, easy enough to find and put it in later. So :+1:.

repo: codeforamerica/snap-it-up


text:
Use Streetmix logo.

We'd been using this very simple text mark for a little while now, this just brings it into the application.
Old:

New:

Are we going to change logo? I haven't review SVG image content, but change looks fine.
:rocket:

We've used this in a bunch of other places already! Putting it in the application will hopefully be better for consistency.

authors: louh magul

repo: codeforamerica/streetmix


text:
revised design of vote detail page #75

revised design from pull request #96
But still not mobile ready. Will fix it later.
related issue #75

Thanks! @thasarito

authors: rapee thasarito

repo: codeforthailand/politician-directory


text:
[global] Implement import order

https://github.com/jfmengels/eslint-plugin-import-order

Done

repo: codefoundries/UniversalRelayBoilerplate


text:
Failed to run freestyle step: Save Terraform statefile to Codefresh context; caused by NonZeroExitCodeError: Container for step title: Save Terraform statefile to Codefresh context, step type: freestyle, operation: Freestyle step failed with exit code: 2

Running freestyle step: Save Terraform statefile to Codefresh context Pulling image codefresh/cli:0.8.54 Digest: sha256:ff73c256681cbf6e6ef8304748d50e6f54903096b7dad9116554694b5fedfbb6 Status: Image is up to date for codefresh/cli:0.8.54 fetch http://dl-cdn.alpinelinux.org/alpine/v3.6/main/x86_64/APKINDEX.tar.gz fetch http://dl-cdn.alpinelinux.org/alpine/v3.6/community/x86_64/APKINDEX.tar.gz (1/2) Installing bash (4.3.48-r1) 7  0% [                                          ]8Executing bash-4.3.48-r1.post-install (2/2) Installing make (4.2.1-r0) 7 75% [###############################           ]87 97% [######################################### ]87100% [##########################################]8Executing busybox-1.26.2-r9.trigger OK: 86 MiB in 35 packages Context: eks-install-dydx patched [SYSTEM] Error: Failed to run freestyle step: Save Terraform statefile to Codefresh context; caused by NonZeroExitCodeError: Container for step title: Save Terraform statefile to Codefresh context, step type: freestyle, operation: Freestyle step failed with exit code: 2

This happens whenever I try to run the step following this tutorial
https://codefresh.io/kubernetes-tutorial/getting-started-amazon-eks/

Is the same to me:
[SYSTEM] Error: Failed to run freestyle step: Save Terraform statefile to Codefresh context; caused by NonZeroExitCodeError: Container for step title: Save  Terraform statefile to Codefresh context, step type: freestyle, operation: Freestyle step failed with exit code: 2

I don't know if this is related, but I found this in all steps make: *** No rule to make target 'codefresh-save-tfstate'.  Stop.
Maybe the pipeline is not finding the scripts files.

I got as far as the last step (codefresh-add-cluster.sh)
[SYSTEM] Error: Failed to run freestyle step: Add EKS cluster to Codefresh account; caused by NonZeroExitCodeError: Container for step title: Add EKS cluster to Codefresh account, step type: freestyle, operation: Freestyle step failed with exit code: 2

However it looks like the Codefresh API has changed. I changed the scripts locally to change the API key header from x-access-token to Authorization but it looks like the call to add a new cluster to codefresh has now been removed...?

Hi there @leomeneguzzi did you find a solution for using eks-installer or built a custom solution?

@kevinfaguiar Sorry, I didn't find a solution for this. I cannot help :(

Having the same issue here... Also following the steps at https://codefresh.io/kubernetes-tutorial/getting-started-amazon-eks/

authors: FaisalAbid javajenks kevinfaguiar leomeneguzzi lpossamai

repo: codefresh-io/eks-installer


text:
Bug: Filter in routes doesn't work if user input url in different case

If I use filter in routes, for example: $routes->get('admin/home/x1', 'admin\home: x1', ['filter' => 'fil']);
its works if user open the url with lowercase letters only (https://localhost/public/admin/home/x1),
but if the user opens the url in uppercase letters (https://localhost/public/admin/HOmE/X1) then the filter does not work and this I think to be unsafe. sorry for my bad english

No feedback and stale. Closing.

authors: adidesember paulbalandan


text:
Fix Sqlite Table:createTable()

This PR fixes https://github.com/codeigniter4/CodeIgniter4/issues/6205
The indexes need to be checked for missing fields on composite keys.
Checklist:

[x] Securely signed commits
[x] Component(s) with PHPDoc blocks, only if necessary or adds value
[x] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

@MGatner check and see if this resolves https://github.com/codeigniter4/CodeIgniter4/issues/6205 for you.

I will test when I'm on desktop. I actually ended up refactoring that whole table out of the repo that had the issue but I can pull an older version of the code that should still have the bug.

Looking at your code It looks like if you want to restore it to your original table then you will have to add the composite index back.
    private function createActions(): void
    {
        $this->forge = \Config\Database::forge('tests');

        $this->forge->dropTable('actions', true);
        // Actions
        $fields = [
            'category'    => ['type' => 'varchar', 'constraint' => 63],
            'name'        => ['type' => 'varchar', 'constraint' => 63],
            'uid'         => ['type' => 'varchar', 'constraint' => 63],
            'class'       => ['type' => 'varchar', 'constraint' => 63, 'null' => true],
            'input'       => ['type' => 'varchar', 'constraint' => 63, 'null' => true],
            'role'        => ['type' => 'varchar', 'constraint' => 63, 'default' => ''],
            'icon'        => ['type' => 'varchar', 'constraint' => 63, 'default' => ''],
            'summary'     => ['type' => 'varchar', 'constraint' => 255, 'default' => ''],
            'description' => ['type' => 'text', 'null' => true],
            'created_at'  => ['type' => 'datetime', 'null' => true],
            'updated_at'  => ['type' => 'datetime', 'null' => true],
            'deleted_at'  => ['type' => 'datetime', 'null' => true],
        ];

        $this->forge->addField('id');
        $this->forge->addField($fields);

        $this->forge->addKey('name');
        $this->forge->addKey('uid');
        $this->forge->addKey(['category', 'name']);
        $this->forge->addKey(['deleted_at', 'id']);
        $this->forge->addKey('created_at');

        $this->forge->createTable('actions');
    }

    public function up(): void
    {
        $this->forge->dropColumn('actions', [
            'description',
            'summary',
            'category',
            'icon',
            'role',
            'class',
        ]);
    }

    public function down(): void
    {
        $this->forge->addColumn('actions', [
            'class'       => ['type' => 'varchar', 'constraint' => 63, 'null' => true],
            'role'        => ['type' => 'varchar', 'constraint' => 63, 'default' => ''],
            'icon'        => ['type' => 'varchar', 'constraint' => 63, 'default' => ''],
            'category'    => ['type' => 'varchar', 'constraint' => 63, 'default' => ''],
            'summary'     => ['type' => 'varchar', 'constraint' => 255],
            'description' => ['type' => 'text', 'constraint' => 255],
        ]);

        // NEED TO ADD BACK COMPOSITE INDEX
        $this->forge->addKey(['category', 'name'])->processIndexes('actions');
    }

This would require a new method to Forge:
    /**
     * Executes Sql to add or remove indexes
     */
    public function processIndexes(string $table): bool
    {
        if (! empty($this->keys)) {
            $this->fields = array_flip(array_map(
                static fn ($columnName) => $columnName->name,
                $this->db->getFieldData($this->db->DBPrefix . $table)
            ));

            for ($i = 0, $sqls = $this->_processIndexes($this->db->DBPrefix . $table), $c = count($sqls); $i < $c; $i++) {
                if ($this->db->query($sqls[$i]) === false) {
                    return false;
                }
            }
        }

        $this->reset();

        return true;
    }

Then of course we would need to test it for all DBMS.
I'm not sure why this method doesn't exist. There is a dropKey() method that executes but the addKey() method does not. I know its because sometimes the keys are created with tables while others do not. This really ought to be fixed.

I agree in the key methods. I think needing to re-add the key is fine, the important part is that the alteration doesn't cause a database error. If you've tests that with this PR then I will proceed with the review.

I've found there are a lot of issues related to this. In MySql, if you have a composite index named category_name and then you drop the column category, the index drops the field category and retains name.. and still assumes the index name category_name. So then if you try and add the index category_name back in then it throws an error. Things really get messy when dealing with auto increment primary keys.
I think we need to just leave it up to the developer to properly manage the indexes. In your example you should have dropped any related indexes before dropping the columns.. then add back those indexes after recreating the columns.
One thing I realized though is that there is no method to drop primary keys and no method to create indexes on existing tables. I'm working on a separate PR to add these methods. At least in MySql dropKey() doesn't work with primary key and requires a different SQL statement.
The code in this PR should stop the error you were reporting, however it still doesn't provide a way to add the index back.

The code in this PR should stop the error you were reporting, however it still doesn't provide a way to add the index back.

Should this PR be closed because this seems jiust a workaround to prevent the error?

No this PR is still necessary.  It fixes a bug. SQLite uses a Table class unlike the rest of the DBMS. It basically copies the table but on recreate doesn't create fields that were dropped. The problem was that it was trying to create indexes for fields that were dropped.
I'm working on another PR that adds the features to create indexes after table creation. I can add that in here if you would rather. I was going to submit that one to 4.3

The problem was that it was trying to create indexes for fields that were dropped.

I got it. Can you add test code?

I'm working on another PR that adds the features to create indexes after table creation. I can add that in here if you would rather. I was going to submit that one to 4.3

#4814
It is an enhancement. Please send to 4.3.

All this tracks with what we've experienced around Forge and the different drivers. I'm happy to have this in as a bugfix, and look forward to what other changes you have for these other issues. Hopefully we can get some DB people to review.

Added testDropColumnDropCompositeKey()

authors: MGatner kenjis sclubricants


text:
docs: fix request.rst

Description

fix incorrect descriptions
add a missing method
some improvements

Checklist:

[x] Securely signed commits
[] Component(s) with PHPDoc blocks, only if necessary or adds value
[] Unit testing, with >80% coverage
[x] User guide updated
[x] Conforms to style guide

It seems Request should be abstract.
And CURLRequest is an HTTP client, not an HTTP request or doing too much things.


text:
refactor: improve error message for HTTP.invalidHTTPProtocol

Description
See https://forum.codeigniter.com/showthread.php?tid=81584&pid=408038#pid408038

Valid versions are obvious, so not needed.

Checklist:

[x] Securely signed commits
[] Component(s) with PHPDoc blocks, only if necessary or adds value
[] Unit testing, with >80% coverage
[x] User guide updated
[x] Conforms to style guide

This requires translation updates. So I changed to 4.4 branch.


text:
refactor: [BaseConnection] change @property-read annotations to just @property

Description
I'm not sure what's the intention of putting these @property-read annotations. Please enlighten me. However, these means the properties are "readonly by annotation", hence they cannot be reassigned later on. But we reassign them in several places across the database connection so I think the intention does not meet the actual practice. Found this because intelephense is complaining of "Cannot modify readonly property ...".
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

I think these magic properties cannot be written.
We cannot write like $db->DBDriver = 'SQLite3, but we can read $db->DBDriver.
So they should be @property-read.

The @property-read and @property-write variants MAY be used to indicate "magic" properties that can only be read or written.
See https://docs.phpdoc.org/3.0/guide/references/phpdoc/tags/property.html

Okay, I took a look again. These are not magic properties. These are protected properties. But we allow reading their values thru BaseConnection::__get(), that's why we have the annotations.

If they are protected properties, they are not  readonly.
So the "Cannot modify readonly property ..." message is wrong.

authors: kenjis paulbalandan


text:
chore: reuse github workflows

reuse github workflows
update compare.yml
install rector and update rector.php
update coding style

References:

https://github.com/Nyholm/roave-bc-check-docker/issues/32#issuecomment-1384621015

Detected last version: v1.0.0
Comparing from b965e7f51a1575a1fce49035886c7fbdaf87190a to dd441eb6c04ff2194698361032b4881a91c0ce6d...
<warning>No composer.lock file present. Updating dependencies to latest instead of installing from lock file. See https://getcomposer.org/install for more information.</warning>
Loading composer repositories with package information
GitHub API limit (60 calls/hr) is exhausted, could not fetch https://api.github.com/repos/codeigniter4/CodeIgniter4/contents/composer.json?ref=6b2816c7fe7bef75da6be286a16800c4f5d9992d. Create a GitHub OAuth token to go over the API rate limit. You can also wait until 2023-09-07 03:59:45 for the rate limit to reset.

When working with _public_ GitHub repositories only, head to https://github.com/settings/tokens/new?scopes=&description=Composer+on+fv-az395-860+2023-09-07+0259 to retrieve a token.
This token will have read-only permission for public information only.
When you need to access _private_ GitHub repositories as well, go to https://github.com/settings/tokens/new?scopes=repo&description=Composer+on+fv-az395-860+2023-09-07+0259
Note that such tokens have broad read/write permissions on your behalf, even if not needed by Composer.
Tokens will be stored in plain text in "/home/runner/.composer/auth.json" for future use by Composer.
For additional information, check https://getcomposer.org/doc/articles/authentication-for-private-packages.md#github-oauth
Token (hidden):
In AuthHelper.php line 132:

  Could not authenticate against github.com

roave-backwards-compatibility-check:assert-backwards-compatible [--from [FROM]] [--to TO] [--format [FORMAT]] [--install-development-dependencies]

Error: Process completed with exit code 255.

https://github.com/codeigniter4/cache/actions/runs/6104869392/job/16567528390?pr=16

I probably screwed this up merging the Dependabot PR. Sorry.

Create a GitHub OAuth token

I have a couple for the CodeIgniter org, let me know if you want me to install in the repo secrets.

authors: MGatner kenjis


text:
Bug: Does not appear to work with MS SQL

PHP Version
8.2.12
CodeIgniter4 Version
4.2.2
Shield Version
v1.0.0-beta.7
Which operating systems have you tested for this bug?
Windows
Which server did you use?
cli-server (PHP built-in webserver)
Database
MS SQL Server 2019
Did you customize Shield?
No, no customisations made.
What happened?
Started with a fresh code igniter installation and connected successfully to a Microsoft SQL Database, connection works and tested ok. Then installed (using composer) Shield. As Soon as installed Shield getting error messages:
Unable to connect to the database.
Main connection [sqlsrv]: [Microsoft][ODBC Driver 17 for SQL Server]TCP Provider: No connection could be made because the target machine actively refused it.
SQLSTATE: 08001, code: 10061
[Microsoft][ODBC Driver 17 for SQL Server]Login timeout expired SQLSTATE: HYT00, code: 0
[Microsoft][ODBC Driver 17 for SQL Server]A network-related or instance-specific error has occurred while establishing a connection to SQL Server. Server is not found or not accessible. Check if instance name is correct and if SQL Server is configured to allow remote connections. For more information see SQL Server Books Online. SQLSTATE: 08001, code: 10061

SQL server detailed are correct despite the error message. As soon as shield is removed the site begins to function again.
Steps to Reproduce
Install Shield with a fresh code igniter install and connect to a SQL server database.
Expected Output
Expecting to see the product work correct.
Anything else?
No response

Shield does not supprot CodeIgniter 4.2.2.

authors: kenjis ollieonline


text:
[Snyk] Upgrade yaml from 1.10.0 to 1.10.2

Snyk has created this PR to upgrade yaml from 1.10.0 to 1.10.2.

:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 2 versions ahead of your current version.
The recommended version was released a day ago, on 2021-03-13.

Release notes

Package name: yaml

1.10.2 - 2021-03-13
Allow for unindented comment after node props (prettier/prettier#10510)

1.10.1 - 2021-03-13This release backports the following non-breaking fixes made during the work on yaml@2 on top of yaml@1.10.0:

Support for __proto__ as mapping key & anchor identifier (#192)
Fix broken TS type for BigInt toggle
Dump long keys properly (#195)
When folding highly indented lines, require at least minContentWidth chars on the first line (#196)
Fix YAML.stringify() for certain null values (#197)
Do not break escaped chars with escaped newlines (#237, awslabs/cdk8s#494)
Set type: "module" within browser/dist/ (#208)
Use CommonJS for the browser endpoints yaml/types & yaml/util (#208)
Always stringify non-Node object keys using explicit notation (#218)
Specify node type of Document.Parsed.contents (#221)
Add missing type for CST Node.rangeAsLinePos (#222)
Prefer literal over folded block scalar when lineWidth=0 is set (#232)
Allow for empty lines after node props (#242)
Update dev dependencies

1.10.0 - 2020-05-16This will probably be the last minor release of yaml@1. I'm aiming to release yaml@2 within a few months; prereleases of that will be published using the next dist-tag on npm. Patch releases for 1.10 may still happen, if necessary.
New Features

Use Rollup for Node.js & browser builds (#165)

This removes most of the internal dist/ paths from the release. If you want/need to use a class or function that is no longer public, please file an issue and we can add it to the exports.
Drop dependency on @ babel/runtime. After this, the package has 0 runtime dependencies. 🎉
Add exports { Alias, Collection, Merge, Node } to 'yaml/types'

Document Schema.createPair() & make its ctx arg optional (#157)
Always indent top-level scalars with lines starting with document markers or % directives (#162)
Use double-space when forcing top-level block scalar indent, for clarity (#162)
Add getNodes(): string[] method to Anchors (#166)
Refactor Jest config, adding tests for compiled dist/ endpoints
Rename & refactor source files. This should have no effect on the results, but lots of stuff moved around

Improved Errors & Warnings

Throw more helpful error when setting Pair.commentBefore incorrectly (#157)
Better errors for bad indents (#169)
Drop incorrect error for flow mapping keys with length > 1024 chars
Add errors for plain scalars that start with reserved indicators
Add more explicit errors for block scalar values with bad indents
Enable log prints during npm start debugging

Improved TypeScript declarations

Fix/simplify export mapping of 'yaml/types' and 'yaml/util'
Fix types, dropping AST.{AstNode,ScalarNode,CollectionNode} (#160)
Add missing toString() methods to AST nodes (#159)
Add directivesEndMarker to Document type (#167)

Bugfixes

Use CommonJS for top-level default-exporting browser endpoints (#163)
docs: Add note about simpleKeys for null values (#170)
Update dev & playground dependencies, including eslint 7 & jest 26
Update yaml-test-suite, fixing errors in remaining invalid-input tests

Add explicit error for block scalars with more-indented leading empty lines (S98Z)
Properly add error for tab-indented map values (4EJS)
Add error for multiline implicit flow sequence pair key (DK4H & ZXT5)
Require space after comma before comment in flow collections (CVW2)
Use mapAsMap: true for out.yaml tests (4FJ6)

    from yaml GitHub release notes

Commit messages

Package name: yaml

4cdcde6 1.10.2
7c0e083 Allow for unindented comment after node props (#242)
8ef0157 1.10.1
6296dae Update links in docs
b1d2b28 Allow for empty lines after node props (Fixes #242)
3e5a640 Satisfy Prettier
bd031cb Update dev dependencies + switch to lockfileVersion 2
9c6e7d0 Use CommonJS for browser endpoints yaml/types & yaml/util (#208)
7ddb18b Prefer literal over folded block scalar when lineWidth=0 is set (#232)
fd817be Update dev dependencies
90cee5f Specify node type of Document.Parsed.contents (#221)
1ff8db7 Add missing type for CST Node.rangeAsLinePos (Fixes #222)
55d7984 Update dev dependencies
94cb60e Always stringify non-Node object keys using explicit notation (#218)
f9fa9c9 Set type: "module" within browser/dist/ (Fixes #208)
0fa01f8 Do not break escaped chars with escaped newlines (awslabs/cdk8s#494)
452b504 Update dev dependencies
dd5c406 When folding highly indented lines, require at least minContentWidth chars on the first line (Fixes #196)
f6f1393 Fix YAML.stringify() for certain null values (#197)
83b0e54 Dump long keys properly (#195)
54dac4f Drop obsolete Travis CI badge from README
4ad5ccf Stop trying to run BrowserStack tests automatically on PRs
44202ee Fix broken TS type for BigInt toggle
dc61b13 Support for __proto__ as mapping key & anchor identifier (#192)

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Coverage remained the same at 97.79% when pulling d284668d916b7acd05e585941a697c15502e7147 on snyk-upgrade-d7f9d1ad65126a43c8d722e4bdc910c0 into 6ab047c1429ca9f01757384740725a0524fe31d5 on master.

authors: codekie coveralls

repo: codekie/openapi-examples-validator


text:
ignore

Before filing a new issue, be sure to check the FAQ for answers to many common questions about Dinghy.

Sorry, Error !

repo: codekitchen/dinghy


text:
Show-hint addon fails to select hint if there is no initially-selected hint

If autocompletion hints displayed without initially-selected hint by passing selectedHint === -1, it will fail with an exception on trying to select a hint:

TypeError: Cannot read property 'className' of undefined
at Widget.changeActive (show-hint.js:342)

I guess it's assuming there's always a selected option. Want to submit a patch?

Thanks!

authors: marijnh stan-z

repo: codemirror/CodeMirror


text:
yjs server can't connect to the new kernel

The yjs and api server still try to connect the previous kernel after activating a new kernel, to reproduce,

create a new runtime
delete the previous runtime
check yjs container logging
check api container logging

The problem got fixed after restart yjs and api containers twice.

Need some help to understand the resolver_yjs.ts and yjs-runtime.ts.

There are some leftover tasks in runtime. I'll work on that.
For now, if a runtime doesn't connect, just delete it and create a new one.

repo: codepod-io/codepod


text:
The real use of the repo

I'm quite skeptical about the working if this project. I want to know more of how things work in here. I really appreciate all the work, but will also be interested to really understand the advantage of this project over our usual Udemy. We've been used-to and it will be very hard for us to forget Udemy.

We bring forth the ability for lecturers/instructors to chose whether to
make their course free or to monetize it. Also for students, we give them
the opportunity to view some of the course material (10%) before enrolling
to the course
El sáb., 5 de noviembre de 2022 10:43 a. m., Joys652 <
@.***> escribió:

I'm quite skeptical about the working if this project. I want to know more
of how things work in here. I really appreciate all the work, but will also
be interested to really understand the advantage of this project over our
usual Udemy. We've been used-to and it will be very hard for us to forget
Udemy.
—
Reply to this email directly, view it on GitHub
https://github.com/coder-Joy/LIQ/issues/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2PRLVYZRVVIJQ4QWA6P5CLWGYT5PANCNFSM6AAAAAARX3EKKM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

repo: coder-Joy/LIQ


text:
Command Pallete Shortcut is not being intercepted on Firefox

code-server version: 1.32.0-310
OS Version: CoreOS build 2025

Description
On Firefox Quantum, Coder does not intercept the command pallete shortcut which is the private browsing shortcut as well on Firefox. Even Alt + KEY refuses to work.
Steps to Reproduce

Open Firefox
Hit the command pallete shortcut

@sr229 Does F5 work? (should still be fixed for the other one)

I don't believe we can override this.

I think folks at Theia found a way around this, worth a look.

To be sure I made a small test on an empty webpage binding keydown and preventing default and stopping propagation on the event but had no luck.
I searched around Theia's GitHub and didn't find anything except this open issue: https://github.com/theia-ide/theia/issues/1696. As mentioned there, F1 is a viable workaround, but it doesn't appear possible to override ctrl+shift+p.

repo: codercom/code-server