Skip to content

Conversation

@raginjason
Copy link
Contributor

This guide came out of a discussion in devcontainers/ci#205. While GitHub has a devcontainer action, nothing like this exists for GitLab CI. This caused a bit of a gap in documentation on how to get advanced devcontainer functionality into GitLab CI. This PR is designed to close that gap.

@raginjason raginjason requested a review from a team as a code owner February 27, 2023 17:04
@raginjason
Copy link
Contributor Author

Tagging @bamurtaugh for review

Copy link
Member

@samruddhikhandale samruddhikhandale left a comment

Choose a reason for hiding this comment

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

This looks great, thank you! ✨

Left some thoughts/comments :)

@bamurtaugh
Copy link
Member

Thanks so much for opening this @raginjason, this looks like a great guide! And thanks @samruddhikhandale for the quick review ✨ - I agree with the suggestions she mentions.

},
"features": {
"ghcr.io/devcontainers/features/aws-cli:1": {},
"ghcr.io/devcontainers-contrib/features/aws-cdk:2": {}
Copy link
Member

Choose a reason for hiding this comment

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

To avoid confusion for folks, it would be nice to point out that devcontainers-contrib is community-maintained (issues on our repos likely wouldn't help get issues in these Features fixed as fast)

Copy link
Contributor Author

@raginjason raginjason Feb 27, 2023

Choose a reason for hiding this comment

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

@joshspicer I've added a note in the paragraph above, does this satisfy your concern? (see 02b35db)

Copy link
Member

@joshspicer joshspicer left a comment

Choose a reason for hiding this comment

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

Very awesome :D

Copy link
Member

@samruddhikhandale samruddhikhandale left a comment

Choose a reason for hiding this comment

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

Nice! ⚡ ⚡

@raginjason
Copy link
Contributor Author

@microsoft-github-policy-service agree company="DataScan"

@raginjason raginjason force-pushed the working-with-gitlab-ci branch from e4842bd to 7e769c3 Compare February 27, 2023 19:25
@raginjason raginjason force-pushed the working-with-gitlab-ci branch from 7e769c3 to 02b35db Compare February 28, 2023 15:08
@samruddhikhandale samruddhikhandale merged commit e57ffdb into devcontainers:gh-pages Feb 28, 2023
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