Add hour and minute for the post permalink.

[felix-fly]

What does it do?

Add hour and minute for the post permalink.

How to test

Edit _config.yml file

permalink: :year/:month:day:hour:minute

Screenshots

Pull request tasks

• Add test cases for the changes.
• Passed the CI test.

[coveralls]

Coverage remained the same at 97.146% when pulling c32b5fa on felix-fly:master into e8fc2ba on hexojs:master.

[coveralls]

Coverage decreased (-0.006%) to 97.141% when pulling 4a7ac1a on felix-fly:master into e8fc2ba on hexojs:master.

[felix-fly]

Can this be merged?

[tomap]

It looks good, but no unit tests?

[tomap]

https://github.com/hexojs/hexo/blob/master/test/scripts/filters/post_permalink.js#L84

Adding one does not seem complex, based upon existing ones

Can you do one, or do you need help?

it('post_title', () => {
    hexo.config.permalink = ':year/:month/:day/:hour/:minute/:post_title/';

    return Post.insert({
      source: 'sub/2015-05-06-my-new-post.md',
      slug: '2015-05-06-my-new-post',
      title: 'My New Post',
      date: moment('2015-05-06 12:13')
    }).then(post => {
      postPermalink(post).should.eql('2015/05/06/12/13/my-new-post/');
      hexo.config.permalink = PERMALINK;
      return Post.removeById(post._id);
    });
  });

That should be a start :)

[curbengh]

Just a note that when hour and minute is enabled,

permalink: :year/:month:day:hour:minute

but front-matter of a post doesn't have hour+minute, the permalink will default them to zero, e.g. "2015/05/06/00/00/my-new-post/".

[curbengh]

CI benchmark doesn't look great, but I believe it's more likely due to Travis is under heavy load (on weekend).

[SukkaW]

LGTM!

[SukkaW]

CI benchmark doesn't look great, but I believe it's more likely due to Travis is under heavy load (on weekend).

@curbengh
We might check https://www.traviscistatus.com/ and triger a rebuild when there are less Active Linux Builds for Open Source projects.