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

CI: add branches filter #1130

Merged
merged 9 commits into from
Oct 10, 2022
Merged

Conversation

6543
Copy link
Member

@6543 6543 commented Aug 25, 2022

No description provided.

@6543
Copy link
Member Author

6543 commented Aug 25, 2022

expect: exec for pull
get: 204 !

@anbraten
Copy link
Member

similar to #1111 at least that was my intention in there as well

@6543
Copy link
Member Author

6543 commented Aug 26, 2022

yes but this looks like it is broken ... on gitea it seems to work :O

@anbraten
Copy link
Member

anbraten commented Oct 7, 2022

@6543 I added some root.when filters and replaced the branches filter.

@woodpecker-bot
Copy link
Collaborator

woodpecker-bot commented Oct 7, 2022

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

@anbraten anbraten marked this pull request as ready for review October 7, 2022 08:14
@codecov-commenter
Copy link

codecov-commenter commented Oct 9, 2022

Codecov Report

Base: 44.36% // Head: 44.36% // No change to project coverage 👍

Coverage data is based on head (0c8897d) compared to base (12ff74c).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1130   +/-   ##
=======================================
  Coverage   44.36%   44.36%           
=======================================
  Files         133      133           
  Lines        9762     9762           
=======================================
  Hits         4331     4331           
  Misses       5174     5174           
  Partials      257      257           

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

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@6543 6543 added the build CI pipeline related label Oct 9, 2022
@6543 6543 merged commit f9dd651 into master Oct 10, 2022
@6543 6543 deleted the ci_only-exec-on-default-and-release-branch branch October 10, 2022 10:49
@6543
Copy link
Member Author

6543 commented Oct 14, 2022

@anbraten we don't get any builds lately ... I'll revert it partially and we need a more clever solution

6543 added a commit that referenced this pull request Oct 14, 2022
@6543
Copy link
Member Author

6543 commented Oct 14, 2022

f56d081

6543 added a commit that referenced this pull request Oct 14, 2022
@anbraten
Copy link
Member

@6543 Yip, saw that as well. I will try to find a solution for the root level when filters. Currently they are reporting pending state back although they are not running at all

simmstein pushed a commit to simmstein/woodpecker that referenced this pull request Dec 27, 2022
Co-authored-by: Anbraten <anton@ju60.de>
simmstein pushed a commit to simmstein/woodpecker that referenced this pull request Dec 27, 2022
simmstein added a commit to simmstein/woodpecker that referenced this pull request Dec 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build CI pipeline related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants