Skip to content

Conversation

@Le-Caignec
Copy link
Contributor

No description provided.

@Le-Caignec Le-Caignec requested review from Copilot, gfournieriExec and zguesmi and removed request for Copilot October 29, 2025 16:10
@Le-Caignec Le-Caignec changed the title fix: update create and deploy scripts for consistency with versioning fix: deployment scripts for consistency with versioning Oct 29, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the Graph deployment configuration by modifying the subgraph naming scheme and version labeling. The changes introduce a new DEPLOY_ENV environment variable for flexible environment-based deployments and update default version labels.

Key changes:

  • Introduces DEPLOY_ENV environment variable with / as default for flexible subgraph naming
  • Updates subgraph name from poco to poco-v5 in create and deploy scripts
  • Changes default version label from bellecour/poco-v5 to dev in the deploy script

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Le-Caignec Le-Caignec self-assigned this Oct 29, 2025
@zguesmi
Copy link
Member

zguesmi commented Oct 30, 2025

Could you add how to deploy on different environments (temp, staging) in the related readme section.
Commands with env variables.

Le-Caignec and others added 2 commits October 30, 2025 15:12
Co-authored-by: Zied Guesmi <[email protected]>
Co-authored-by: Zied Guesmi <[email protected]>
@Le-Caignec Le-Caignec merged commit 7b1c57c into main Oct 30, 2025
5 checks passed
@Le-Caignec Le-Caignec deleted the fix/deployment-script branch October 30, 2025 14:22
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.

3 participants