Skip to content

Conversation

@jzbahrai
Copy link
Contributor

Summary | Résumé

Failure when setting up the admin devcontainer, this PR fixes that

Note: Ajoutez vous une nouvelle page par Articles GC? Assurez vous d'ajouter le chemin dans les règles WAF.

Test instructions | Instructions pour tester la modification

Sequential steps (1., 2., 3., ...) that describe how to test this change. This
will help a developer test things out without too much detective work. Also,
include any environmental setup steps that aren't in the normal README steps
and/or any time-based elements that this requires.


Étapes consécutives (1., 2., 3., …) qui décrivent la façon de tester la
modification. Elles aideront les développeurs à faire des tests sans avoir à
jouer au détective. Veuillez aussi inclure toutes les étapes de configuration
de l’environnement qui ne font pas partie des étapes normales dans le fichier
README et tout élément temporel requis.

@github-actions
Copy link

Copy link
Contributor

@smcmurtry smcmurtry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm getting the following failure when trying to build the dev container:
3.714 E: Unable to locate package eza

And I'm able to build the dev container in the main branch without any errors.

@jzbahrai jzbahrai force-pushed the task/fix-dev branch 2 times, most recently from f25bc0a to 3ccbf28 Compare November 28, 2025 19:28
procps \
sudo \
tldr \
tealdeer \
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙇

Copy link
Member

@jimleroyer jimleroyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, let's make sure cypress still work after the removal of libgconf-2-4. It seems that this is a dependency for older version of cypress, so that might still work after its removal here, as we would have carried this dep since then.

Copy link
Contributor

@smcmurtry smcmurtry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is now building for me and the tests are passing. Nice!

@smcmurtry
Copy link
Contributor

Although I think this did break the copilot chat integration... is there any way to fix that?

@jzbahrai jzbahrai marked this pull request as draft December 10, 2025 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants