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.

Sign up for GitHub

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

Jump to bottom

Bug: multipipeline runs on more than .yml files #271

Closed
anbraten opened this issue Aug 20, 2021 · 0 comments · Fixed by #279
Closed

Bug: multipipeline runs on more than .yml files #271

anbraten opened this issue Aug 20, 2021 · 0 comments · Fixed by #279

Comments

@anbraten
Copy link
Member

If I have a file called .tests.yml.inactive inside my multi-pipeline folder, it gets executed as well. IMO only specific files should be executed like .yml, .yaml, whatever else is possible to be used.
@anbraten anbraten mentioned this issue Aug 28, 2021
Merged
@6543 6543 closed this as completed in #279 Aug 30, 2021
6543 pushed a commit that referenced this issue Aug 30, 2021
@anbraten
Fix filter pipeline config files (#279)
34cfabb 
closes #271 

- filter pipeline config folders for `.yml` and `.yaml` files
- improve `fetchConfig` tests
- update remote mock and correct wrong folder name `mock` => `mocks` to match package name
- fix: return correct filename for fallback
- improve config loading by checking if folder or not before sending api call
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: filter pipeline config files anbraten/woodpecker
1 participant
@anbraten