Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deploy option to use local chains code. Fixes BT-11787 #1246

Merged
merged 2 commits into from
Nov 18, 2024

Conversation

marius-baseten
Copy link
Contributor

@marius-baseten marius-baseten commented Nov 14, 2024

🚀 What

  • This allows faster dev loops when running chains with local docker.
  • It also allows more integrity of the e2e integration test, since now the repo code (instead main/HEAD) is used in the CI.

💻 How

🔬 Testing

@marius-baseten marius-baseten force-pushed the marius/chains-local-src branch 2 times, most recently from 50f05a9 to 471b1c5 Compare November 14, 2024 23:12
Copy link
Contributor

@tyranitar tyranitar 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! Thanks for doing this.

truss-chains/truss_chains/definitions.py Outdated Show resolved Hide resolved
@marius-baseten marius-baseten force-pushed the marius/chains-local-src branch from c77fd2c to a9c194b Compare November 15, 2024 20:23
@marius-baseten marius-baseten force-pushed the marius/chains-local-src branch from a9c194b to 6e086c7 Compare November 18, 2024 23:09
@marius-baseten marius-baseten merged commit 9e8c458 into main Nov 18, 2024
5 checks passed
@marius-baseten marius-baseten deleted the marius/chains-local-src branch November 18, 2024 23:37
@marius-baseten marius-baseten changed the title Deploy option to use local chains code. Deploy option to use local chains code. Fixes BT-11787 Nov 20, 2024
Copy link

linear bot commented Nov 20, 2024

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