Re: Pull request for ExtUtils::Depends submitted to GNOME Gitlab



I believe the MR was branched off current master, so no rebasing should be 
necessary.

My opinion of the merge commit issue is that it's Ok to have merge commits, 
or to not have them (so long as the commit author is preserved, which oddly 
other project maintainers can be very casual about).

My advice would be to use whichever is most convenient for the maintainer.

Best regards,
Ed

-----Original Message----- 
From: TorstenSchönfeld
Sent: Saturday, March 23, 2019 7:28 PM
To: gtk-perl-list gnome org
Subject: Re: Pull request for ExtUtils::Depends submitted to GNOME Gitlab

On Wed, 2019-03-20 at 17:33 -0700, Brian Manning wrote:

Ed J submitted a pull request for ExtUtils::Depends to the GNOME
GitLab server;

https://gitlab.gnome.org/GNOME/perl-extutils-depends/merge_requests/1

Basically, the changes cause EU::D to quote paths if they have spaces
in them, and to bump the version number of EU::D up so it's higher
than a version of EU::D that's included with a release Gtk-Perl
version 1 that's currently on CPAN
(https://metacpan.org/release/Gtk-Perl).

My questions are:
1) Does anyone see any problems merging this, and

The changes look good to me.

2) Are we using merge commits or fast forward merges for merging
merge requests?

Up until now we've been using fast-forward merge commits, but if you
make sure to rebase the branch onto master before merging, I wouldn't
be opposed to merge commits either if you prefer them.

-Torsten

_______________________________________________
gtk-perl-list mailing list
gtk-perl-list gnome org
https://mail.gnome.org/mailman/listinfo/gtk-perl-list 



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