Skip to content
This repository has been archived by the owner on Feb 8, 2019. It is now read-only.

Conversation

peckpeck
Copy link
Member

@peckpeck
Copy link
Member Author

I will let both @amousset and @fanf look at this one.

@amousset
Copy link
Member

@fanf @peckpeck OK for me.

@peckpeck
Copy link
Member Author

You may wait for the package manager to be merge first. To make sure this works.

@fanf
Copy link
Member

fanf commented Mar 2, 2017

It is ok for me, to, sorry forgot to tell.

@amousset
Copy link
Member

amousset commented Mar 2, 2017

@peckpeck Do you want to test it again before merging?

"version": "${plugin-version}",
"build-date": "${maven.build.timestamp}",
"build-commit": "${commit-id}",
"jar-files": [ "datasources.jar" ],
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@peckpeck This now needs to be a full path

@peckpeck
Copy link
Member Author

peckpeck commented Mar 2, 2017

Commit modified

@peckpeck peckpeck force-pushed the arch_10102/package_datasource_plugin_properly branch from 263153f to 42aaea3 Compare March 2, 2017 14:59
@Normation-Quality-Assistant

OK, merging this PR

@Normation-Quality-Assistant Normation-Quality-Assistant merged commit 42aaea3 into Normation:branches/rudder/4.1 Mar 2, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
4 participants