Skip to content

Merge patch 3 #1

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 28 commits into from
Feb 24, 2024
Merged

Merge patch 3 #1

merged 28 commits into from
Feb 24, 2024

Conversation

@Carbrex
Copy link
Owner

@Carbrex Carbrex commented Feb 24, 2024

No description provided.

Carbrex added 21 commits February 19, 2024 23:56
@Carbrex
cd into the dir where lila-docker is located
3953fed
@Carbrex
Trying to add lila PR checkout functionality in setup() function
889e174
@Carbrex
Try fixing expected Result<(), Error>, found () error
f61b58c
@Carbrex
Remove semicolon to fix another return error
96b3b88
@Carbrex
Remove return Ok();
4a91d8b
@Carbrex
Move remaining part to else in main.rs setup
8f7fdd6
@Carbrex
Add environment variable printing and handle missing PR number
9832e72
@Carbrex
Refactor setup function to handle missing environment variable GITPOD…
e28d292 
…_GIT_PR_NUMBER
@Carbrex
Get Environment variable from nested Gitpod_workspace_context
f3319b6
@Carbrex
Add serde_json to dependencies
3e03100
@Carbrex
Try fixing match error
cd18d57
@Carbrex
Merge branch 'patch-4' into patch-3
07b6e3a
@Carbrex
Add print statements to check where the functionality broke
bdb4047
@Carbrex
Update workspace context print statement
4663b01
@Carbrex
Update lila_pr_no type to String
c01dac7
@Carbrex
Remove debug print statements in main.rs
14ae4c6
@Carbrex
Refactor load_lila_pr_no function
b583096
@Carbrex
Add return
177f296
@Carbrex
Merge patch-4 to patch-3
436ebff
@Carbrex
Add gitpod_checkout_pr function to load and checkout a specific pull …
99a8c78 
…request
@Carbrex
Merge changes
7721aaf
@Carbrex Carbrex merged commit b7bce78 into patch-4 Feb 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Carbrex