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.

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

config.properties should be in main/resources #33

Open
nelenkov opened this issue Sep 21, 2017 · 1 comment
Open

config.properties should be in main/resources #33

nelenkov opened this issue Sep 21, 2017 · 1 comment

Comments

@nelenkov
Copy link

Otherwise it is not automatically picked up by the war task. Alternatively added a war rule to included it, but better to follow the default layout.

e.g.,

fidouaf/src/main/resources/org/ebayopensource/fidouaf/res/config.properties

Also, any objections replacing config with a structured file (YAML, etc) so that all configurable properties (AID list, origin, multiple app facets, etc.) can be configured in one place?

ngs-mtech pushed a commit to ngs-mtech/UAF that referenced this issue Feb 28, 2018
@JazzDeben
Copy link

JazzDeben commented Apr 5, 2018

I noticed that too, my resolution was to change the POM with this:
<resources>
  <resource>
    <directory>src/main/java</directory>
   <includes>
    <include>**/*.properties</include>
   </includes>
  </resource>
</resources>
At the start of the <build> tag.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants