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

Speed up pipelines #2127

Merged
merged 2 commits into from
Aug 7, 2023
Merged

Speed up pipelines #2127

merged 2 commits into from
Aug 7, 2023

Conversation

qwerty287
Copy link
Contributor

@qwerty287 qwerty287 commented Aug 7, 2023

  • Use workflow-global conditions to also skip the clone step for docs and web workflows
  • never run test and web for pushes (they're running for PRs and every change we do must pass checks on PRs, so they're unnecessary)

#1910

@qwerty287 qwerty287 added the build CI pipeline related label Aug 7, 2023
@qwerty287 qwerty287 requested a review from a team August 7, 2023 08:13
@qwerty287 qwerty287 merged commit 516bc47 into woodpecker-ci:main Aug 7, 2023
@qwerty287 qwerty287 deleted the speed-workflows branch August 7, 2023 08:58
@qwerty287 qwerty287 mentioned this pull request Aug 7, 2023
qwerty287 added a commit that referenced this pull request Aug 7, 2023
Some steps were not longer executed and some `when` conditions are not
longer necessary. Follow-up #2127
@qwerty287
Copy link
Contributor Author

qwerty287 commented Aug 7, 2023

Hmm unfortunately, I forgot that the checks are required on Github and if we skip them completely no PR can pass.
How can we easily fix this without having to run the useless workflows? -> #2129

@qwerty287 qwerty287 mentioned this pull request Aug 7, 2023
6543 pushed a commit that referenced this pull request Aug 7, 2023
Since it does not work this is mainly a revert of #2127
@6543
Copy link
Member

6543 commented Aug 7, 2023

got reverted: #2129

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build CI pipeline related skip-changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants