savannah-hackers-public
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Savannah-hackers-public] Initial changes for adding the Git homepag


From: Bob Proulx
Subject: Re: [Savannah-hackers-public] Initial changes for adding the Git homepage source code web browsing option.
Date: Wed, 3 Jan 2024 17:26:13 -0700

Hi Ayushman,

Ayushman Tripathi wrote:
> Hello! I'm currently working towards implementing the option for the Git
> webpage repository.
>
> This patch specifically covers the initial frontend changes.

Thank you for submitting this patch suggestion for review.  As I look
at this so far I don't see the added controls for using a git web page
yet.  What I see is an overloading of the current controls.

> +if ($project->CanUse("homepage") && $project->CanUse("git")) {

That's an insufficient combination.  Many current projects already use
git for their source code and many of them won't want to change their
web page version control at this time.  But having this control logic
would force them to make a web page version control change.

Also it would happen immediately for everyone.  Everyone all at once.
But it will take time for projects that want to change to get ready to
make the change.

Unless I am misunderstanding what is being suggested.  If so please
correct me and get me straightened out.

I believe we will need to implement a new row in the Select Features
administration panel.  Implement a new database field to hold the new
values.  As discussed today in the meeting we think that there should
be an enumeration for different possible version control systems.

There are only three magic numbers.  Zero, One, and Many.  If we are
moving past One then we should set up for the Many case.  I think we
should set up an enumeration of possible version control systems to
include the current CVS and the new incoming GIT.  Let's not include
any more unless more are needed but it allows expansion to more in
the future.  It would allow an SVN project to be set up for SVN for
web pages as well.  And the same for HG.  And so on.  But let's focus
on just the two CVS and GIT for the moment in order to keep things
manageable initially.

This will require altering the schema of the database to include the
new data fields.  And of course new code to interact with those
fields.  I think you will need to ask for help in doing those parts of
the task.  That's going to be expected.

Let me applaud you for getting the standalone setup up and running!
That's really good!  That's better than a lot of people have been able
to do recently.  Good job!

Since you have recently done this I think the best thing to do next is
to get the documentation updated which covers this part of the task.
Just by itself that is a big deal.  Let's make it a better thing by
getting the documentation for doing it updated so that it helps
everyone doing this next.

Good progress! :-)
Bob



reply via email to

[Prev in Thread] Current Thread [Next in Thread]