Summary

Always replace {branch} and replace / with _

Review Request #11758 — Created July 29, 2021 and submitted Aug. 25, 2021, 4:05 p.m.

Information

Owner

misery

Repository

rbintegrations

Branch

release-2.0.x

Bugs

Depends On

Reviewers

Groups

rbintegrations

People

Description

{branch} shouldn't be a Git-only variable. Any version system
can use this.

If a repository name contains / we need to convert it into
another character because Jenkins do not allow / as a job name.
So let's use _ as a replacement.

Testing Done

Configured a jenkins integration with job name {branch}_{repository}.
The repository name contains multiple /. Started a review request
and saw that it correctly started a job with replaced names.

Commits

Summary	ID
Always replace {branch} and replace / with _ {branch} shouldn't be a Git-only variable. Any version system can use this. If a repository name contains / we need to convert it into another character because Jenkins do not allow / as a job name. So let's use _ as a replacement.	1269e9c6316f5ecd154dff675cca74d820f41ada

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Status: Closed (submitted)

Change Summary:

Pushed to release-2.0.x (b52a06f)