Skip to content
This repository was archived by the owner on Jun 11, 2025. It is now read-only.

feat: new deployment super task with env vars names aligned on gateway #455

Merged
merged 1 commit into from
Apr 4, 2025

Conversation

jatZama
Copy link
Member

@jatZama jatZama commented Apr 3, 2025

No description provided.

@jatZama jatZama requested a review from a team as a code owner April 3, 2025 16:10
@cla-bot cla-bot bot added the cla-signed label Apr 3, 2025
@jatZama jatZama force-pushed the deploymentSuperTask branch from 24a4a0a to f73f889 Compare April 3, 2025 16:15
Copy link

@eudelins-zama eudelins-zama left a comment

Choose a reason for hiding this comment

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

LGTM but let's just wait for @0xawaz opinion regarding the dockerfile issue

Copy link
Contributor

@PacificYield PacificYield left a comment

Choose a reason for hiding this comment

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

There are a few changes that could be done to improve readability and avoid repeating the same code.

Copy link
Collaborator

@tawadaa tawadaa left a comment

Choose a reason for hiding this comment

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

lgtm, did few changes on docker compose part, which will be automated and run in teh CI from now on

@jatZama jatZama force-pushed the deploymentSuperTask branch from 9b6fee7 to 50c54c2 Compare April 4, 2025 09:38
chore: update dockerfiles

chore: refactor checkIsHardhatSigner

chore(deploy): fix env file loading

chore(deploy): fix prettier

fix(ci): remove unnecessary checkout tokens in GPU CI, which are broken since update of repo secrets
@jatZama jatZama force-pushed the deploymentSuperTask branch from 92e33f0 to bd8403a Compare April 4, 2025 09:50
@jatZama jatZama requested a review from PacificYield April 4, 2025 09:52
@jatZama jatZama dismissed PacificYield’s stale review April 4, 2025 09:53

Already resolved all of them.

@jatZama jatZama merged commit d7abf21 into main Apr 4, 2025
15 checks passed
@jatZama jatZama deleted the deploymentSuperTask branch April 4, 2025 09:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants