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

fix(CI/CD): correct CI variable. #19886

Merged
merged 2 commits into from
Jun 5, 2022
Merged

fix(CI/CD): correct CI variable. #19886

merged 2 commits into from
Jun 5, 2022

Conversation

appleboy
Copy link
Member

@appleboy appleboy commented Jun 4, 2022

default value is true for CI variable

see https://docs.drone.io/pipeline/environment/reference/ci/

Signed-off-by: Bo-Yi Wu appleboy.tw@gmail.com

default value is true for CI variable

see https://docs.drone.io/pipeline/environment/reference/ci/

Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>
@appleboy appleboy added this to the 1.17.0 milestone Jun 4, 2022
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Jun 4, 2022
@lunny
Copy link
Member

lunny commented Jun 4, 2022

@appleboy It's amazing the code stay there 5 years. Did that affect anything?

@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 Jun 5, 2022
@appleboy
Copy link
Member Author

appleboy commented Jun 5, 2022

@lunny I try to release command in the local machine but an error occurred in the following code:

cd $(DIST); for file in `find /build -type f -name "*"`;

It is missing folder /build.

@lunny lunny merged commit dadcaa4 into go-gitea:main Jun 5, 2022
@appleboy appleboy deleted the ci branch June 5, 2022 11:11
zjjhot added a commit to zjjhot/gitea that referenced this pull request Jun 5, 2022
* giteaofficial/main:
  Add alt text to logo (go-gitea#19892)
  Limit max-height of CodeMirror editors for issue comment and wiki (go-gitea#18271)
  Implement http signatures support for the API (go-gitea#17565)
  Increment tests time out from 40m to 50m because sometimes the machine is slow (go-gitea#19887)
  fix(CI/CD): correct CI variable. (go-gitea#19886)
  Fix typo (go-gitea#19889)
  Fixing wrong paging when filtering on the issue dashboard (go-gitea#19801)
  Move `/info` outside authorization (go-gitea#19888)
  Fix order by parameter (go-gitea#19849)
  Exclude Archived repos from Dashboard Milestones (go-gitea#19882)
  use exact search instead of fuzzy search for branch filter dropdown (go-gitea#19885)
mehm8128 added a commit to traPtitech/gitea that referenced this pull request Aug 16, 2022
AbdulrhmnGhanem pushed a commit to kitspace/gitea that referenced this pull request Aug 24, 2022
default value is true for CI variable

see https://docs.drone.io/pipeline/environment/reference/ci/

Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>

Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
mehm8128 added a commit to traPtitech/gitea that referenced this pull request Dec 28, 2022
mehm8128 added a commit to traPtitech/gitea that referenced this pull request Jan 1, 2023
mehm8128 added a commit to traPtitech/gitea that referenced this pull request Mar 26, 2023
@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. topic/deployment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants