juneau-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "John D. Ament" <johndam...@apache.org>
Subject Re: Sample Codes on Documents
Date Wed, 24 May 2017 01:45:24 GMT
I started putting this together - best to give it a try and see what can be
improved

https://cwiki.apache.org/confluence/display/JUNEAU/Contributing+for+Non-committers?flashId=-2057206781

John

On Tue, May 23, 2017 at 9:31 PM James Bognar <jamesbognar@gmail.com> wrote:

> How did you create it though?
>
> On Tue, May 23, 2017 at 9:30 PM John D. Ament <johndament@apache.org>
> wrote:
>
> > James,
> >
> > An email just came through from github with some instructions, for a
> test I
> > was doing for infra.  Do those instructions look like enough?
> >
> > John
> >
> > On Tue, May 23, 2017 at 9:26 PM James Bognar <jamesbognar@gmail.com>
> > wrote:
> >
> > > Yes, I'm apparently unfamiliar with pull requests.  Can someone more
> > > familiar with git document how to use them?
> > >
> > >
> > > On Tue, May 23, 2017 at 8:01 PM John D. Ament <johndament@apache.org>
> > > wrote:
> > >
> > > > Hi James,
> > > >
> > > > We should discuss further.  With github integration specifically,
> pull
> > > > requests are pretty normal.  They have some benefits - we can track
> on
> > > list
> > > > who created it and the commit history retains the original author
> name.
> > > > This process has become the industry norm.
> > > >
> > > > I've raised an infra issue over the fact that we didn't receive a
> > > > notification of this PR.  Though ideally we should have JIRAs
> covering
> > > > changes whenever possible.
> > > >
> > > > John
> > > >
> > > > On Tue, May 23, 2017 at 2:04 PM James Bognar <
> > > james.bognar@salesforce.com>
> > > > wrote:
> > > >
> > > > > Yep.  I think that's all he needs.  He can just send me patch files
> > > > > attached to dev@juneau emails.
> > > > >
> > > > > On Tue, May 23, 2017 at 2:02 PM, Craig Russell <
> apache.clr@gmail.com
> > >
> > > > > wrote:
> > > > >
> > > > > > I think it is appropriate for these messages to be on dev@juneau.
> > > > > >
> > > > > > My point is: what else does Raphi need to do? Doesn't his pull
> > > request
> > > > > > give you everything you need to commit his changes to the main
> > repo?
> > > > > Isn't
> > > > > > that how pull requests work?
> > > > > >
> > > > > > Craig
> > > > > >
> > > > > > > On May 23, 2017, at 7:31 AM, James Bognar <
> > > > james.bognar@salesforce.com
> > > > > >
> > > > > > wrote:
> > > > > > >
> > > > > > > Hi Craig...
> > > > > > >
> > > > > > > That message was meant for Raphi.  If he has changes he
wants
> to
> > > make
> > > > > to
> > > > > > > the website, he can send me a patch file and I'll commit
it for
> > > him.
> > > > > > >
> > > > > > > On Tue, May 23, 2017 at 10:27 AM, Craig Russell <
> > > > apache.clr@gmail.com>
> > > > > > > wrote:
> > > > > > >
> > > > > > >> Hi James,
> > > > > > >>
> > > > > > >> Maybe I don't understand your request, or don't understand
> git.
> > > > > > >>
> > > > > > >> The pull request has the patch included. If you click
on the
> > > "typos
> > > > > and
> > > > > > >> errors" link or the 29fc3be link the diff opens up.
And then
> you
> > > can
> > > > > > view
> > > > > > >> the whole file, comment, etc.
> > > > > > >>
> > > > > > >> I'm not set up to directly commit from the pull request
page,
> > but
> > > > you
> > > > > > >> should be able to.
> > > > > > >>
> > > > > > >> Is some other work flow better?
> > > > > > >>
> > > > > > >> Craig
> > > > > > >>
> > > > > > >>> On May 23, 2017, at 7:05 AM, James Bognar <
> > > > > james.bognar@salesforce.com
> > > > > > >
> > > > > > >> wrote:
> > > > > > >>>
> > > > > > >>> Hi Raphi,
> > > > > > >>>
> > > > > > >>> If you have any changes, can you attach a git patch
file and
> > I'll
> > > > > > >>> review/commit it for you?  I think that's the approach
our
> team
> > > is
> > > > > > going
> > > > > > >> to
> > > > > > >>> start using for code submissions until developers
get
> committer
> > > > role.
> > > > > > >>>
> > > > > > >>> On Tue, May 23, 2017 at 6:48 AM, Raphi D Lee <
> > yidonguk@gmail.com
> > > >
> > > > > > wrote:
> > > > > > >>>
> > > > > > >>>> Thank you John, Craig.
> > > > > > >>>>
> > > > > > >>>> I’ve created new pull request on github.
> > > > > > >>>>
> > > > > > >>>> https://github.com/apache/incubator-juneau-website/pull/1
<
> > > > > > >>>> https://github.com/apache/incubator-juneau-website/pull/1>
> > > > > > >>>>
> > > > > > >>>> Thank you again.
> > > > > > >>>>
> > > > > > >>>>
> > > > > > >>>> Raphi D Lee, yidonguk@gmail.com
> > > > > > >>>>
> > > > > > >>>>
> > > > > > >>>>
> > > > > > >>>>
> > > > > > >>>>> 2017. 5. 23. 오전 5:35, Craig Russell
<apache.clr@gmail.com>
> > 작성:
> > > > > > >>>>>
> > > > > > >>>>> Thanks for that, John.
> > > > > > >>>>>
> > > > > > >>>>> On the "source code" page
> > > > > http://juneau.apache.org/#sourceCode.html
> > > > > > it
> > > > > > >>>> does have a link "GIT Repository for this website"
> > > > > > >>>> https://git-wip-us.apache.org/repos/asf?p=incubator-juneau-
> > > > > > website.git
> > > > > > >>>> that I totally missed.
> > > > > > >>>>>
> > > > > > >>>>> Craig
> > > > > > >>>>>
> > > > > > >>>>>> On May 22, 2017, at 1:13 PM, John D.
Ament <
> > > > johndament@apache.org
> > > > > >
> > > > > > >>>> wrote:
> > > > > > >>>>>>
> > > > > > >>>>>> Hi Raphi,
> > > > > > >>>>>>
> > > > > > >>>>>> Actually the website is on github as
well -
> > > > > > >>>>>> https://github.com/apache/incubator-juneau-website
-
> please
> > > > note
> > > > > > that
> > > > > > >>>> the
> > > > > > >>>>>> branch is called "asf-site"
> > > > > > >>>>>>
> > > > > > >>>>>> John
> > > > > > >>>>>>
> > > > > > >>>>>> On Mon, May 22, 2017 at 4:10 PM Craig
Russell <
> > > > > apache.clr@gmail.com
> > > > > > >
> > > > > > >>>> wrote:
> > > > > > >>>>>>
> > > > > > >>>>>>> Hi Raphi,
> > > > > > >>>>>>>
> > > > > > >>>>>>>
> > > > > > >>>>>>>> On May 22, 2017, at 2:48 AM,
Raphi D Lee <
> > > yidonguk@gmail.com>
> > > > > > >> wrote:
> > > > > > >>>>>>>>
> > > > > > >>>>>>>> Hi All,
> > > > > > >>>>>>>>
> > > > > > >>>>>>>> There’s no (mirror) repository
on Github for juneau
> > website.
> > > > > > >>>>>>>> Is there any way to create
pull request without it?
> > > > > > >>>>>>>> Please give me a hint.
> > > > > > >>>>>>>
> > > > > > >>>>>>> Excellent question.
> > > > > > >>>>>>>
> > > > > > >>>>>>> I'd like the developers to respond
what the best way is
> to
> > > > > receive
> > > > > > >>>> patches
> > > > > > >>>>>>> to the web site.
> > > > > > >>>>>>>
> > > > > > >>>>>>> Craig
> > > > > > >>>>>>>
> > > > > > >>>>>>>>
> > > > > > >>>>>>>>
> > > > > > >>>>>>>> Raphi D Lee, yidonguk@gmail.com
> > > > > > >>>>>>>>
> > > > > > >>>>>>>>
> > > > > > >>>>>>>>
> > > > > > >>>>>>>>
> > > > > > >>>>>>>>> 2017. 5. 22. 오전 5:26,
Craig Russell <
> > apache.clr@gmail.com>
> > > > 작성:
> > > > > > >>>>>>>>>
> > > > > > >>>>>>>>> Hi Raphi,
> > > > > > >>>>>>>>>
> > > > > > >>>>>>>>> It's great that you have
become involved so quickly as
> a
> > > > > > developer
> > > > > > >> in
> > > > > > >>>>>>> the project.
> > > > > > >>>>>>>>>
> > > > > > >>>>>>>>>
> > > > http://www.apache.org/foundation/how-it-works.html#developers
> > > > > <
> > > > > > >>>>>>>
> > > http://www.apache.org/foundation/how-it-works.html#developers>
> > > > > > >>>>>>>>>
> > > > > > >>>>>>>>> To get started, can you
create a pull request for your
> > > > changes?
> > > > > > >> Then
> > > > > > >>>> a
> > > > > > >>>>>>> committer can update the repository.
> > > > > > >>>>>>>>>
> > > > > > >>>>>>>>>
> > > > http://www.apache.org/foundation/how-it-works.html#committers
> > > > > <
> > > > > > >>>>>>>
> > > http://www.apache.org/foundation/how-it-works.html#committers>
> > > > > > >>>>>>>>>
> > > > > > >>>>>>>>> Keep up the work on the
project and in due time the
> > project
> > > > > will
> > > > > > >> vote
> > > > > > >>>>>>> on offering you commit privileges.
> > > > > > >>>>>>>>>
> > > > > > >>>>>>>>> Regards,
> > > > > > >>>>>>>>>
> > > > > > >>>>>>>>> Craig
> > > > > > >>>>>>>>>
> > > > > > >>>>>>>>>> On May 20, 2017, at
11:32 PM, Raphi D Lee <
> > > > yidonguk@gmail.com
> > > > > > >>>> <mailto:
> > > > > > >>>>>>> yidonguk@gmail.com>> wrote:
> > > > > > >>>>>>>>>>
> > > > > > >>>>>>>>>> Okay, I’ll check
it over. :)
> > > > > > >>>>>>>>>>
> > > > > > >>>>>>>>>> Thanks
> > > > > > >>>>>>>>>>
> > > > > > >>>>>>>>>> Raphi D Lee, yidonguk@gmail.com
> > > > > > >>>>>>>>>>
> > > > > > >>>>>>>>>>
> > > > > > >>>>>>>>>>
> > > > > > >>>>>>>>>>
> > > > > > >>>>>>>>>>> 2017. 5. 21. 오후
3:11, James Bognar <
> > > > > > james.bognar@salesforce.com>
> > > > > > >>>> 작성:
> > > > > > >>>>>>>>>>>
> > > > > > >>>>>>>>>>> Hey Raphi,
> > > > > > >>>>>>>>>>>
> > > > > > >>>>>>>>>>> I'm going to have
to defer to one of the mentors on
> how
> > > > that
> > > > > > >> works
> > > > > > >>>>>>> for new
> > > > > > >>>>>>>>>>> committers.  I
believe you get a username once you've
> > > > signed
> > > > > > and
> > > > > > >>>>>>> emailed
> > > > > > >>>>>>>>>>> your Apache Contributor
License Agreement.  Then it's
> > > just
> > > > > your
> > > > > > >>>> apache
> > > > > > >>>>>>>>>>> account.
> > > > > > >>>>>>>>>>>
> > > > > > >>>>>>>>>>>
> > > > > > >>>>>>>>>>> On Sun, May 21,
2017 at 2:04 AM, Raphi D Lee <
> > > > > > yidonguk@gmail.com
> > > > > > >>>>>>> <mailto:yidonguk@gmail.com>
<mailto:yidonguk@gmail.com
> > > > <mailto:
> > > > > > >>>>>>> yidonguk@gmail.com>>>
wrote:
> > > > > > >>>>>>>>>>>
> > > > > > >>>>>>>>>>>> Hey.. James
> > > > > > >>>>>>>>>>>>
> > > > > > >>>>>>>>>>>> Authentication
is needed to commit yo the master
> > branch.
> > > > > > >>>>>>>>>>>> Could you tell
me how can I have an username for it?
> > > > > > >>>>>>>>>>>> I am sorry
to bother you.
> > > > > > >>>>>>>>>>>>
> > > > > > >>>>>>>>>>>> Thanks
> > > > > > >>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>
> > > > > > >>>>>>>>>>>> Raphi D Lee,
yidonguk@gmail.com <mailto:
> > > > yidonguk@gmail.com>
> > > > > > >>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>> 2017. 5.
21. 오후 2:52, James Bognar <
> > > > > > >> james.bognar@salesforce.com
> > > > > > >>>>>>> <mailto:james.bognar@salesforce.com>>
작성:
> > > > > > >>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>> Yep...commit
it to the master branch.
> > > > > > >>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>> On Sun,
May 21, 2017 at 1:50 AM, Raphi D Lee <
> > > > > > >> yidonguk@gmail.com
> > > > > > >>>>>>> <mailto:yidonguk@gmail.com>
> > > > > > >>>>>>>>>>>> <mailto:yidonguk@gmail.com
<mailto:
> yidonguk@gmail.com
> > >
> > > > > > <mailto:
> > > > > > >>>>>>> yidonguk@gmail.com <mailto:yidonguk@gmail.com>>>>
wrote:
> > > > > > >>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>> Thank
you James,
> > > > > > >>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>> Could
you tell me What “deliver changes” means?
> Push
> > > to
> > > > > git
> > > > > > >>>> origin?
> > > > > > >>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>> Raphi
D Lee, yidonguk@gmail.com <mailto:
> > > > > yidonguk@gmail.com>
> > > > > > >>>>>>> <mailto:yidonguk@gmail.com <mailto:yidonguk@gmail.com>>
> > > > > > >>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
2017. 5. 21. 오후 2:32, James Bognar <
> > > > > > >>>> james.bognar@salesforce.com
> > > > > > >>>>>>> <mailto:james.bognar@salesforce.com>
<mailto:
> james.bognar@
> > > > > > >>>> salesforce.com
> > > > > > >>>>>>> <mailto:james.bognar@salesforce.com>>>
작성:
> > > > > > >>>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
Hi Raphi,
> > > > > > >>>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
These all look great!  Thanks for catching them!
> > > > > > >>>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
Line 812 on overview-summary.html is okay though.
> > > > There
> > > > > > >>>> *should*
> > > > > > >>>>>>> be a
> > > > > > >>>>>>>>>>>>>> ws()
method on SerializerGroupBuilder.  I just
> added
> > > it.
> > > > > > >> Thanks
> > > > > > >>>>>>> for
> > > > > > >>>>>>>>>>>>>> finding
that as well.
> > > > > > >>>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
You can deliver these changes.
> > > > > > >>>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
On Sun, May 21, 2017 at 12:47 AM, Raphi D Lee <
> > > > > > >>>> yidonguk@gmail.com
> > > > > > >>>>>>> <mailto:yidonguk@gmail.com>
<mailto:yidonguk@gmail.com
> > > > <mailto:
> > > > > > >>>>>>> yidonguk@gmail.com>>
> > > > > > >>>>>>>>>>>>>> <mailto:yidonguk@gmail.com
<mailto:
> > yidonguk@gmail.com
> > > >
> > > > > > >> <mailto:
> > > > > > >>>>>>> yidonguk@gmail.com <mailto:yidonguk@gmail.com>>
<mailto:
> > > > > > >>>> yidonguk@gmail.com
> > > > > > >>>>>>> <mailto:yidonguk@gmail.com>
<mailto:yidonguk@gmail.com
> > > > <mailto:
> > > > > > >>>>>>> yidonguk@gmail.com>>>>>
wrote:
> > > > > > >>>>>>>>>>>>>>>
Hi all,
> > > > > > >>>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
I’ve found some typos and errors in sample codes
> in
> > > > > > overview
> > > > > > >>>>>>> document
> > > > > > >>>>>>>>>>>>>> and
javadocs while reviewing.
> > > > > > >>>>>>>>>>>>>>>
Please let me know if there’s bad modifications.
> > > > > > >>>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
Thanks,
> > > > > > >>>>>>>>>>>>>>>
Regards
> > > > > > >>>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
Raphi D Lee, yidonguk@gmail.com <mailto:
> > > > > yidonguk@gmail.com
> > > > > > >
> > > > > > >>>>>>> <mailto:yidonguk@gmail.com <mailto:yidonguk@gmail.com>>
> > > > <mailto:
> > > > > > >>>>>>> yidonguk@gmail.com <mailto:yidonguk@gmail.com>
<mailto:
> > > > > > >>>> yidonguk@gmail.com
> > > > > > >>>>>>> <mailto:yidonguk@gmail.com>>>
<mailto:
> > > > > > >>>>>>>>>>>> yidonguk@gmail.com
<mailto:yidonguk@gmail.com>>
> > > > > > >>>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>>
--
> > > > > > >>>>>>>>>>>>>>>
James Bognar
> > > > > > >>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>> --
> > > > > > >>>>>>>>>>>>> James Bognar
> > > > > > >>>>>>>>>>>>
> > > > > > >>>>>>>>>>>>
> > > > > > >>>>>>>>>>>
> > > > > > >>>>>>>>>>>
> > > > > > >>>>>>>>>>> --
> > > > > > >>>>>>>>>>> James Bognar
> > > > > > >>>>>>>>>>
> > > > > > >>>>>>>>>
> > > > > > >>>>>>>>> Craig L Russell
> > > > > > >>>>>>>>> clr@apache.org <mailto:clr@apache.org>
> > > > > > >>>>>>>
> > > > > > >>>>>>> Craig L Russell
> > > > > > >>>>>>> Secretary, Apache Software Foundation
> > > > > > >>>>>>> clr@apache.org http://db.apache.org/jdo
> > > > > > >>>>>>>
> > > > > > >>>>>>>
> > > > > > >>>>>
> > > > > > >>>>> Craig L Russell
> > > > > > >>>>> Secretary, Apache Software Foundation
> > > > > > >>>>> clr@apache.org http://db.apache.org/jdo
> > > > > > >>>>>
> > > > > > >>>>
> > > > > > >>>>
> > > > > > >>>
> > > > > > >>>
> > > > > > >>> --
> > > > > > >>> James Bognar
> > > > > > >>
> > > > > > >> Craig L Russell
> > > > > > >> Secretary, Apache Software Foundation
> > > > > > >> clr@apache.org http://db.apache.org/jdo
> > > > > > >>
> > > > > > >>
> > > > > > >
> > > > > > >
> > > > > > > --
> > > > > > > James Bognar
> > > > > >
> > > > > > Craig L Russell
> > > > > > Secretary, Apache Software Foundation
> > > > > > clr@apache.org http://db.apache.org/jdo
> > > > > >
> > > > > >
> > > > >
> > > > >
> > > > > --
> > > > > James Bognar
> > > > >
> > > >
> > >
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message