Christophe Coevoet

Results 2811 comments of Christophe Coevoet

We should add a temporary test running a subset of the upstream `descriptions_with_comments.feature`, with descriptions only for nodes on which we support them (currently, it is skipped because we don't...

If you don't intend to run `self-update` in your container (which you probably should not if you use a multi-stage build as updating composer should be done by updating the...

This is a known issue in XDebug 3.3+: https://bugs.xdebug.org/view.php?id=2222

Removing this optimization would increase a lot the size of the method, given that most services have the same naming pattern for all "normal" AWS regions. AFAICT, the special cases...

Actually, we already have such sorting in place in https://github.com/async-aws/aws/blob/5bf5e4984ae5aa27f7155d26dd46d35c54fd9240/src/CodeGenerator/src/Generator/ClientGenerator.php#L216-L229 However, the current sorting involves the number of regions in a group, which is why adding a new one often...

My approach there would be to actually configure the `News` page as being matched as active for any concrete news (as those are not rendered anyway). But it is true...

@Nyholm @jderusse do you have any preference about the solution for that ? It is needed to unblock #1861

My investigation of the .NET SDK confirms that their code generator does not have any special code for union shapes, and so the code is generated like for a normal...

@barbieswimcrew what is your preferred approach for the way to support Symfony 8 ? This choice might influence the way I would write the compat layer.