-
Notifications
You must be signed in to change notification settings - Fork 0
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
TIDY enqueue #124
TIDY enqueue #124
Conversation
$filegroups->{$key} = new HTFeed::FileGroup($files,%$val); | ||
my $self = shift; | ||
|
||
if(not defined $self->{filegroups}) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
An exception to the "space after if" rule that seems to be in place elsewhere. Looks like there are some other examples in this file too, also I'm in favor of "space after while" and I see an instance of a while( ...
just below. I think it would be worth doing a search on "if(" and "while(" and aligning the whitespace with the overall standard.
|
||
$self->{directory_files} = []; | ||
my $stagedir = $path || $self->get_staging_directory(); | ||
opendir(my $dh,$stagedir) or croak("Can't opendir $stagedir: $!"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems to be an exception to the "space after param comma" standard, which I also endorse. Ditto my comment about searching for other exceptions, there seem to be some but not a lot.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two comments re spacing conventions I recommend. I like what I see here. APPROVE.
deleting a merged branch |
Trying to clean up the enqueue-related code in order to take care of some TODOs (from https://hathitrust.atlassian.net/wiki/spaces/ETT/pages/2928082947/Potential+ingest+work).
Should be mostly whitespace / reordering with no functional changes.
One commit per tidied file for ease of reviewing, that I'll squash post-review / pre-merge.