Skip to content
This repository has been archived by the owner on Nov 16, 2017. It is now read-only.

fix RF-13169 #18

Closed
wants to merge 1 commit into from
Closed

fix RF-13169 #18

wants to merge 1 commit into from

Conversation

pslegr
Copy link

@pslegr pslegr commented Sep 5, 2013

Fixed, but did not tested, its for Lukas ;)

@lfryc
Copy link
Contributor

lfryc commented Sep 5, 2013

Hey Pavel,

I have two ideas which could be better then this:

  1. reuse VersionBean
  2. or use your apporach and load the version.properties in configuration and in VersionBean from same source
  3. reuse the setup for propagating configuration through arquillian.xml

@lfryc
Copy link
Contributor

lfryc commented Sep 11, 2013

@pslegr could you please follow-up on this pull request?

I'm not saying your solution wouldn't work, I would just prefer to use one principle for all cases.
In this case we already use arquillian.xml as a filtered resource, where settings from pom.xml are propagated.

@pslegr
Copy link
Author

pslegr commented Sep 11, 2013

closed as won'tfix see https://issues.jboss.org/browse/RF-13169

@pslegr pslegr closed this Sep 11, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants