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

add gitpod config #20995

Merged
merged 11 commits into from
Oct 11, 2022
Merged

add gitpod config #20995

merged 11 commits into from
Oct 11, 2022

Conversation

anbraten
Copy link
Contributor

Add a config for https://www.gitpod.io/. This allows users to easily run an online IDE pre-configured to develop and test Gitea by opening https://gitpod.io/#https://github.com/go-gitea/gitea.

Open in Gitpod

@lunny lunny added the type/docs This PR mainly updates/creates documentation label Aug 30, 2022
@silverwind
Copy link
Member

Any benefit in using 2 separate commands instead of our combined make watch?

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Aug 31, 2022
@anbraten
Copy link
Contributor Author

It makes reading logs and manually restarting a bit easier.

Copy link
Member

@silverwind silverwind left a comment

Choose a reason for hiding this comment

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

Seems resonable

@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Aug 31, 2022
.gitpod.yml Outdated Show resolved Hide resolved
@codecov-commenter
Copy link

codecov-commenter commented Sep 1, 2022

Codecov Report

Merging #20995 (23bf01b) into main (04e97b8) will increase coverage by 46.80%.
The diff coverage is n/a.

❗ Current head 23bf01b differs from pull request most recent head 2d6ea49. Consider uploading reports for the commit 2d6ea49 to get more accurate results

@@            Coverage Diff            @@
##           main   #20995       +/-   ##
=========================================
+ Coverage      0   46.80%   +46.80%     
=========================================
  Files         0     1017     +1017     
  Lines         0   138551   +138551     
=========================================
+ Hits          0    64852    +64852     
- Misses        0    65769    +65769     
- Partials      0     7930     +7930     
Impacted Files Coverage Δ
modules/templates/base.go 63.52% <0.00%> (ø)
routers/api/v1/admin/org.go 46.31% <0.00%> (ø)
services/auth/source/oauth2/init.go 42.85% <0.00%> (ø)
routers/private/internal_repo.go 50.00% <0.00%> (ø)
routers/web/user/setting/security/2fa.go 12.70% <0.00%> (ø)
services/task/migrate.go 33.33% <0.00%> (ø)
cmd/web_acme.go 0.00% <0.00%> (ø)
modules/private/manager.go 0.00% <0.00%> (ø)
modules/options/base.go 0.00% <0.00%> (ø)
modules/process/context.go 87.87% <0.00%> (ø)
... and 1007 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Oct 11, 2022
@techknowlogick techknowlogick merged commit 5aceedc into go-gitea:main Oct 11, 2022
@anbraten anbraten deleted the gitpod branch October 11, 2022 16:15
zjjhot added a commit to zjjhot/gitea that referenced this pull request Oct 13, 2022
* upstream/main: (25 commits)
  [skip ci] Updated translations via Crowdin
  Respect user's locale when rendering the date range in the repo activity page (go-gitea#21410)
  Consolidate more CSS colors into variables (go-gitea#21402)
  Add HEAD fix to gitea doctor (go-gitea#21352)
  Contribution guidelines (go-gitea#21425)
  Refactor Gitpod configuration to improve quick spin up of automated dev environments (go-gitea#21411)
  Support instance-wide OAuth2 applications (go-gitea#21335)
  Case-insensitive NuGet symbol file GUID (go-gitea#21409)
  Add generic set type (go-gitea#21408)
  Improve OAuth integration tests (go-gitea#21390)
  Make e-mail sanity check more precise (go-gitea#20991)
  Fix broken link to frontend guidelines in hacking guidelines (go-gitea#21382)
  Use Name instead of DisplayName in LFS Lock (go-gitea#21415)
  [skip ci] Updated translations via Crowdin
  feat(pr review): add more space on mobile (go-gitea#21326)
  Bump `golang.org/x/text` (go-gitea#21412)
  Update gitea.service (go-gitea#21399)
  Do DB update after merge in hammer context (go-gitea#21401)
  add gitpod config (go-gitea#20995)
  Remove cancel button in repo creation page (go-gitea#21381)
  ...
@yardenshoham yardenshoham mentioned this pull request Nov 15, 2022
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/docs This PR mainly updates/creates documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants