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

Rework pipeline messages #4626

Draft
wants to merge 28 commits into
base: main
Choose a base branch
from
Draft

Conversation

qwerty287
Copy link
Contributor

@qwerty287 qwerty287 commented Dec 27, 2024

supersedes #3421

closes #2971

todo

@qwerty287 qwerty287 added the refactor delete or replace old code label Dec 27, 2024
@qwerty287 qwerty287 added the breaking will break existing installations if no manual action happens label Jan 5, 2025
@pat-s
Copy link
Contributor

pat-s commented Jan 6, 2025

@qwerty287 Why is that breaking? Because of the env var change?

@qwerty287
Copy link
Contributor Author

Yes, env var change and it also changes the json for the pipeline model

@woodpecker-bot
Copy link
Collaborator

woodpecker-bot commented Jan 11, 2025

Deploying preview to https://woodpecker-ci-woodpecker-pr-4626.surge.sh

@qwerty287
Copy link
Contributor Author

@woodpecker-ci/maintainers somebody has an idea how to fix the DB tests? https://ci.woodpecker-ci.org/repos/3780/pipeline/24869/38

Besides that, this PR can get first reviews. It still needs testing and the UI is probably not the best.

Also, there are some todos, especially in the bitbucket code. Maybe @langecode you can check this out?

@xoxys
Copy link
Member

xoxys commented Jan 11, 2025

Can you add a bit more context about what has changed and how it fixes the linked issue? IMO it helps to provide a more detailed PR description in addition to the code.

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 refactor delete or replace old code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Environment Variable CI_COMMIT_AUTHOR_EMAIL is wrong
4 participants