+1 for putting it into the repo. I am fine with either putting it into
README.md or linking form there.

On Tue, Sep 11, 2018 at 4:33 PM, Ryan McMahon <mcmellaw...@apache.org>
wrote:

> Kirk - I have a PR open here which has the "Setting up IntelliJ" section
> you've described:
> https://github.com/apache/geode/pull/2456
>
> I would be happy to use your version if you think it is more comprehensive,
> or we can revise my PR to include any details you feel are missing.
>
> Ryan
>
> On Tue, Sep 11, 2018 at 4:15 PM, Jacob Barrett <jbarr...@pivotal.io>
> wrote:
>
> > Put it with the source! BUILDING.md or other file.
> >
> > > On Sep 11, 2018, at 4:11 PM, Kirk Lund <kl...@apache.org> wrote:
> > >
> > > I don't care which location (wiki or part of the readme) but I do have
> > > up-to-date instructions that I could update either location with.
> Maybe a
> > > detailed "Setting up IntelliJ" section on BUILDING.md? Just let me know
> > if
> > > you'd like my version.
> > >
> > >> On Tue, Sep 11, 2018 at 3:36 PM, Jianxia Chen <jche...@apache.org>
> > wrote:
> > >>
> > >> +1 to revise the wiki to link to README.md/BUILDING.md
> > >>
> > >> On Tue, Sep 11, 2018 at 3:22 PM Ryan McMahon <mcmellaw...@apache.org>
> > >> wrote:
> > >>
> > >>> Hi all,
> > >>>
> > >>> I am looking to add more comprehensive instructions for how to bring
> > >> Geode
> > >>> into IntelliJ.  I've written the instructions but am now looking at
> > where
> > >>> to put them.
> > >>>
> > >>> There appears to be duplicate information in these sections of the
> > Geode
> > >>> wiki and the README.md/BUILDING.md in the Geode Git repository.
> > >>>
> > >>>
> > >>> https://cwiki.apache.org/confluence/display/GEODE/
> > >> Getting+Started+for+Geode+Developers
> > >>>
> > >>> https://cwiki.apache.org/confluence/display/GEODE/
> > >> Building+and+Running+Geode+from+Source
> > >>>
> > >>> I'm a fan of one source of truth, but I wanted to see if anyone felt
> > >>> strongly about where the information lives.  I propose we revise the
> > wiki
> > >>> to simply link to the README.md/BUILDING.md and eliminate the
> duplicate
> > >>> information (how to run gradlew, etc).  Any thoughts?
> > >>>
> > >>> Thanks,
> > >>> Ryan
> > >>>
> > >>
> >
>

Reply via email to