Hi Denis, Yes, I will make a PR to update docs in ignite-website repo as well.
That will cover remaining scope of the jira issue. Regards, Saikat On Mon, 16 Nov 2020 at 9:24 PM, Denis Magda <[email protected]> wrote: > Got it. Didn't pay attention that those changes were in the readme.txt > files. > > Are you adjusting the docs on the website as well? I bet you were > submitting a pull-request earlier. > > - > Denis > > > On Mon, Nov 16, 2020 at 6:31 PM Saikat Maitra <[email protected]> > wrote: > > > Hi Denis, > > > > My thoughts are that these README.txt files will be published as part of > > source releases for Ignite Extensions. > > > > Regards, > > Saikat > > > > On Mon, Nov 16, 2020 at 5:28 PM Denis Magda <[email protected]> wrote: > > > > > Hi Saikat, > > > > > > Please go ahead with the merge. Do we need to publish this for the > Ignite > > > 2.9 docs at some point? Guess, after the extensions are released under > > > different names. > > > > > > - > > > Denis > > > > > > > > > On Sun, Nov 15, 2020 at 9:57 AM Saikat Maitra <[email protected] > > > > > wrote: > > > > > > > Hi, > > > > > > > > As discussed I have updated the PR > > > > https://github.com/apache/ignite-extensions/pull/30 > > > > > > > > Please review and share your feedback. > > > > > > > > Regards, > > > > Saikat > > > > > > > > On Mon, Nov 9, 2020 at 6:38 PM Saikat Maitra < > [email protected]> > > > > wrote: > > > > > > > > > Hi Nikolay, > > > > > > > > > > Thank you for reviewing the changes. I have made changes only in > the > > > > > README.txt file and updated the artifactId so that it matches with > > > > pom.xml. > > > > > > > > > > I have also updated the version details in the PR so that it > matches > > > with > > > > > our release version. > > > > > > > > > > https://github.com/apache/ignite-extensions/pull/30 > > > > > > > > > > With respect to naming convention I was thinking we would not be > > > required > > > > > to change the groupId and we will only change the artifactId > similar > > to > > > > > spring-boot-autoconfigure-ext module. > > > > > > > > > > > > > > > > > > > > > > > > > https://github.com/apache/ignite-extensions/blob/master/modules/spring-boot-autoconfigure-ext/pom.xml#L33 > > > > > > > > > > Please review and let me know your thoughts. > > > > > > > > > > Regards, > > > > > Saikat > > > > > > > > > > > > > > > > > > > > > > > > > On Mon, Nov 9, 2020 at 12:32 AM Nikolay Izhikov < > [email protected] > > > > > > > > wrote: > > > > > > > > > >> Hello, Saikat. > > > > >> > > > > >> As far as I can see you changed artifactId of the extensions not > > only > > > > >> docs. > > > > >> > > > > >> Do we have an agreement to name each extension as > > > «{extension-name}-ext» > > > > >> like «ignite-camel-ext» or similar? > > > > >> We don’t have much extensions release, so maybe it will be better > to > > > > have > > > > >> naming like > > > > >> > > > > >> groupId=org.apache.ignite.extensions > > > > >> artifactId=ignite-camel > > > > >> > > > > >> What do you think? > > > > >> > > > > >> > > > > >> > 9 нояб. 2020 г., в 05:36, Saikat Maitra < > [email protected]> > > > > >> написал(а): > > > > >> > > > > > >> > Hi, > > > > >> > > > > > >> > I have raised a PR for the following issue. > > > > >> > > > > > >> > Jira : https://issues.apache.org/jira/browse/IGNITE-12951 > > > > >> > PR : https://github.com/apache/ignite-extensions/pull/30 > > > > >> > > > > > >> > This is an initial PR in Ignite Extensions repo. I will work on > > > > another > > > > >> PR > > > > >> > in ignite repo for the remaining changes. > > > > >> > > > > > >> > Regards, > > > > >> > Saikat > > > > >> > > > > >> > > > > > > > > > >
