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

Drop "WOODPECKER_WEBHOOK_HOST" env var and adjust docs #3969

Merged
merged 7 commits into from
Jul 25, 2024

Conversation

qwerty287
Copy link
Contributor

  • drop WOODPECKER_WEBHOOK_HOST
  • remove refs to WOODPECKER_DEV_OAUTH_HOST and WOODPECKER_DEV_GITEA_OAUTH_URL
  • and some docs fixes: remove forgejo warning, refer to our own plugins if possible

@qwerty287 qwerty287 added the breaking will break existing installations if no manual action happens label Jul 24, 2024
@qwerty287 qwerty287 added this to the 3.0.0 milestone Jul 24, 2024
@qwerty287 qwerty287 requested a review from a team July 24, 2024 15:32
@woodpecker-bot
Copy link
Collaborator

woodpecker-bot commented Jul 24, 2024

Deployment of preview was successful: https://woodpecker-ci-woodpecker-pr-3969.surge.sh

@qwerty287 qwerty287 merged commit 111eb7e into woodpecker-ci:main Jul 25, 2024
7 checks passed
@qwerty287 qwerty287 deleted the fix-docs branch July 25, 2024 15:17
qwerty287 added a commit that referenced this pull request Jul 25, 2024
@qwerty287 qwerty287 mentioned this pull request Jul 25, 2024
@woodpecker-bot woodpecker-bot mentioned this pull request Jul 25, 2024
1 task
@6543 6543 changed the title Fix dropped env vars Drop "WOODPECKER_WEBHOOK_HOST" env var and adjust docs Aug 31, 2024
6543 added a commit to 6543-forks/woodpecker that referenced this pull request Sep 5, 2024
Co-authored-by: 6543 <6543@obermui.de>
@woodpecker-bot woodpecker-bot mentioned this pull request Sep 8, 2024
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking will break existing installations if no manual action happens
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants