New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move Driver modules into separate files #1

Closed
wants to merge 2 commits into
base: dist
from

Conversation

Projects
None yet
3 participants
@reneeb
Contributor

reneeb commented Feb 5, 2015

No description provided.

@mlawren

This comment has been minimized.

Owner

mlawren commented Feb 5, 2015

Thanks for the pull request Renée! Unfortunately I didn't tell you
about how I manage my branches, and so it is difficult for me to merge
this request as it is. Here is how I do things:

devel: where development happens

devel-dist: automatically generated from the devel branch using a
script I run locally that basically calls "make manifest dist" and
then "git commit".  Hence the MANIFEST file is always automatically
populated.

master: only ever merge from devel when stable releases are made

master-dist: only ever merge from devel-dist when stable releases 
are made

I don't know how the results of the CPAN request challenge are
recorded. I can either cherry-pick your 71819b6 commit onto devel, or
you could re-work the pull request against the devel branch, dropping
the second commit. Let me know which is best?

@mlawren

This comment has been minimized.

Owner

mlawren commented Feb 5, 2015

Whoops - I got the master-dist branch name wrong. It is actually "dist" as you already know.

@mlawren mlawren closed this Mar 6, 2015

@mlawren

This comment has been minimized.

Owner

mlawren commented Mar 6, 2015

I've merged the single commit onto the newly named 'dev' branch, and I'll close this pull request now. Thanks for the help - and if you need the favour returned somewhere let me know.

@reneeb

This comment has been minimized.

Contributor

reneeb commented Mar 6, 2015

Thanks. Sorry for the silence. I was too busy the last month.

@neilb

This comment has been minimized.

neilb commented Mar 6, 2015

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment