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

feat: Add devcontainer and update dev docs #4683

Merged
merged 1 commit into from
May 15, 2023
Merged

Conversation

juliusknorr
Copy link
Member

@juliusknorr juliusknorr commented May 14, 2023

  • Add devcontainer setup to have a fully ready dev environment on github codespaces available
    Screenshot 2023-05-14 at 11 38 58
    Screenshot 2023-05-14 at 12 52 27

  • Update development docs to clarify the different options

    • Codespaces
    • nextcloud-docker-dev: Local single container
    • nextcloud-docker-dev: full setup
    • Local nextcloud setup

@cypress
Copy link

cypress bot commented May 14, 2023

Passing run #912 ↗︎

0 15 0 0 Flakiness 0

Details:

Merge 352e66f into 9bffe51...
Project: Deck Commit: 26cd2c2913 ℹ️
Status: Passed Duration: 02:39 💡
Started: May 14, 2023 11:01 AM Ended: May 14, 2023 11:04 AM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

Signed-off-by: Julius Härtl <jus@bitgrid.net>
@max-nextcloud
Copy link
Contributor

🤯 This is really mind blowing.

Thanks for looking into it. I imagine this will lower the barrier to entry a LOT!

Copy link
Contributor

@max-nextcloud max-nextcloud left a comment

Choose a reason for hiding this comment

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

I had a quick look at the code and the docs. Seems good to me.

Also tried it out by clicking the branch name on top of this PR and then opening it in a code space. Took a while to spin up and compile the JS but then it was working just fine.

Still trying to grasp the implications of this... :)

@juliusknorr juliusknorr merged commit 513ad35 into main May 15, 2023
@juliusknorr juliusknorr deleted the enh/devcontainer branch May 15, 2023 06:26
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.

2 participants