Skip to content

Add development containers support #2379

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

Merged
merged 1 commit into from
May 26, 2025
Merged

Conversation

katallaxie
Copy link
Contributor

@katallaxie katallaxie commented May 2, 2025

Describe your changes

The changes here add support for development containers to make it easy to spin-up a development environment.

To test this works with most of the tools the go run cmd/spicedb/main.go was run to run the server locally and mage -l.

@katallaxie katallaxie requested a review from a team as a code owner May 2, 2025 06:07
@github-actions github-actions bot added the area/docs Affects docs or metadata (e.g. README) label May 2, 2025
@katallaxie
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@katallaxie katallaxie force-pushed the devcontainers branch 2 times, most recently from f6889fa to 9155b04 Compare May 2, 2025 08:32
vroldanbet
vroldanbet previously approved these changes May 2, 2025
Copy link
Contributor

@vroldanbet vroldanbet left a comment

Choose a reason for hiding this comment

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

Great contribution! LGTM!

"customizations": {
"vscode": {
"extensions": [
"golang.go"
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: maybe worth adding the spicedb extension? https://marketplace.visualstudio.com/items?itemName=authzed.spicedb-vscode

Also perhaps docker extension?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Happy to do so. I wanted to keep the surface slim, but these are helpful.

@vroldanbet
Copy link
Contributor

@katallaxie 👋🏻 are we ready to move forward with this PR? in that case, all we are missing is CLA signature

Copy link

github-actions bot commented May 26, 2025

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@katallaxie
Copy link
Contributor Author

recheck

authzedbot added a commit to authzed/cla that referenced this pull request May 26, 2025
@vroldanbet vroldanbet added this pull request to the merge queue May 26, 2025
Merged via the queue into authzed:main with commit 46d7356 May 26, 2025
41 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators May 26, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/docs Affects docs or metadata (e.g. README)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants