Skip to content

remove pointless README file >> master #2

Merged
merged 2 commits into from Apr 5, 2013

2 participants

@nickboldt
JBoss Tools member

No description provided.

nickboldt added some commits Apr 5, 2013
@nickboldt nickboldt JBDS-2490 add jbdevstudio/com.jboss.jbds.central.discovery/ plugin an…
…d fix jbosstools/org.jboss.tools.central.discovery/ so it's in synch w/ latest in SVN; update READMEs to reflect refactored paths and add extra poms to make things easier to build
8d55e89
@nickboldt nickboldt remove pointless README file 2088417
@nickboldt nickboldt merged commit 2088417 into jbosstools:master Apr 5, 2013
@maxandersen
JBoss Tools member

could we please get an informative readme.md instead of one that starts of with things that are not usable by anyone without VPN ?

JBoss Tools member

Yes. Would you like more than one readme? One for "build this locally" and another for "use what's been already built and published to download.jboss.org" ?

JBoss Tools member

I don't understand why all these sub-site specific info is needed here.

Just need to know about -Djboss.discovery.directory.url and then in the subfolders can be details for common used urls for jbosstools or devstudio. No need to mix them all together IMO.

JBoss Tools member

I like to write things down so that when you later come asking me where stuff is, I can point you at the document. :)

Also, you're forgetting about jboss.discovery.site.url, which can be anything regardless of the value of jboss.discovery.directory.url.

JBoss Tools member

This README.md is to be informative - long list of urls in the root doesn't help on that. anyways, I can grok it better now and i'll do a PR to split up so overview and implementation details are not mixed together.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.