Skip to content

Add workflow-job dependency#72

Merged
FlorianVeaux merged 5 commits into
pipelinefrom
florian/add_pipeline_deps
Jun 15, 2020
Merged

Add workflow-job dependency#72
FlorianVeaux merged 5 commits into
pipelinefrom
florian/add_pipeline_deps

Conversation

@FlorianVeaux

@FlorianVeaux FlorianVeaux commented Jun 10, 2020

Copy link
Copy Markdown
Contributor

Note: Target of this PR is the "pipeline" branch, not master.

This PR adds the jenkins BOM to make management of dependencies easier. It also adds a dependency to workflow-job and bumps the minimum jenkins version.

The jenkins version specified here is the minimum that supports TaskListenerDecorator that will be used in a future PR.

@FlorianVeaux
FlorianVeaux requested a review from sarah-witt June 10, 2020 14:11
Comment thread pom.xml Outdated
@FlorianVeaux

FlorianVeaux commented Jun 10, 2020

Copy link
Copy Markdown
Contributor Author

Updated docs as well

@github-actions github-actions Bot added the documentation Documentation related changes label Jun 15, 2020
@FlorianVeaux
FlorianVeaux merged commit 282d238 into pipeline Jun 15, 2020
@FlorianVeaux
FlorianVeaux deleted the florian/add_pipeline_deps branch June 15, 2020 13:40
FlorianVeaux added a commit that referenced this pull request Jun 17, 2020
* Add workflow-job dependency

* Update version

* Reuse jenkins.version constant

* Bumpt to 2.164.x

* Set min version to 2.164.1
FlorianVeaux added a commit that referenced this pull request Jun 18, 2020
* Add workflow-job dependency (#72)

* Add workflow-job dependency

* Update version

* Reuse jenkins.version constant

* Bumpt to 2.164.x

* Set min version to 2.164.1

* Make ClientFactory mockable

* Add workflow-job dependency

* Update version

* Reuse jenkins.version constant

* Bumpt to 2.164.x

* Collect pipeline logs

* Set min version to 2.164.1

* Remove Serializable (it already is)

* Add tests

* Fix tests

* Remove debug logging

* Address review

* Update src/test/java/org/datadog/jenkins/plugins/datadog/logs/LogCollectionTest.java

Co-authored-by: Sarah Witt <[email protected]>

Co-authored-by: Sarah Witt <[email protected]>
@FlorianVeaux FlorianVeaux added the changelog/Added Added features results into a minor version bump label Aug 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/Added Added features results into a minor version bump documentation Documentation related changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants