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

Add TestPipelineName to procBuilder_test.go #461

Merged
merged 2 commits into from
Oct 19, 2021
Merged

Add TestPipelineName to procBuilder_test.go #461

merged 2 commits into from
Oct 19, 2021

Conversation

ctbur
Copy link
Contributor

@ctbur ctbur commented Oct 19, 2021

Split from #455

@ctbur ctbur mentioned this pull request Oct 19, 2021
Merged
@6543 6543 added the tests related to tests or other things CI check before merge label Oct 19, 2021
@6543 6543 added this to the 0.15.0 milestone Oct 19, 2021
@6543 6543 enabled auto-merge (squash) October 19, 2021 07:13
@6543 6543 disabled auto-merge October 19, 2021 07:20
@6543 6543 merged commit fd8e0b2 into woodpecker-ci:master Oct 19, 2021
6543 pushed a commit that referenced this pull request Oct 19, 2021
@ctbur
Add TestPipelineName to procBuilder_test.go (#461) (#455)
5861995 
Backport #461 

Add test for a bug where pipelines that use `depends_on` were not built in multi-pipeline builds. The problem is that pipelines names keep a leading `'/'` when the pipeline path does not have a trailing `'/'`.
@ctbur ctbur deleted the pipeline-name-test branch October 19, 2021 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@6543 6543 6543 approved these changes

Assignees
No one assigned
Labels
skip-changelog tests related to tests or other things CI check before merge
Projects
None yet
Milestone
0.15.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@ctbur @6543