Aaron Sikes FellowMD

  • Joined on
FellowMD commented on pull request jekyll/jekyll-compose#24
@FellowMD

@parkr how we lookin? Should I merge?

@FellowMD
  • @FellowMD 7c85fa7
    Return consistent types in ArgParser
FellowMD commented on issue samg/timetrap#63
@FellowMD

I think this is a very common action. Requiring a user to learn the scripting facilities in order to change the name of a timesheet makes the app f…

FellowMD commented on issue samg/timetrap#69
@FellowMD

@samg This issue can be closed as it's been implemented and merged in

@FellowMD
  • @FellowMD c5857d3
    Fix references to posts in page commands
@FellowMD
-t overshadowed by jekyll built-in -t
FellowMD commented on pull request jekyll/jekyll-compose#24
@FellowMD

Thanks for all the feedback!

@FellowMD
  • @FellowMD bac5ae0
    Address line notes - minor changes only
FellowMD commented on pull request jekyll/jekyll-compose#24
@FellowMD

I was planning on having the unified init_with_program, but it seemed to cause problems with the plugin loading. I'll inline them.

FellowMD commented on pull request jekyll/jekyll-compose#24
@FellowMD

This was a copied literal. I'll remove it

FellowMD commented on pull request jekyll/jekyll-compose#24
@FellowMD

will do

FellowMD commented on pull request jekyll/jekyll-compose#24
@FellowMD

Dunno, I guess I was just checking if I could. It seemed a little silly to indent the whole class two extra tab stops for no reason. It's also a li…

FellowMD commented on pull request jekyll/jekyll-compose#24
@FellowMD

The previous implementation used args.shift, so this seemed equivalent. I'll use args.join that definitely seems better.

FellowMD commented on pull request jekyll/jekyll-compose#24
@FellowMD

The diffs here aren't that informative. Just looking at the new code should be fine when you get a chance.

FellowMD commented on pull request jekyll/jekyll-compose#24
@FellowMD

Would you prefer I squash these?

FellowMD commented on pull request jekyll/jekyll-compose#24
@FellowMD

Fixes #23

FellowMD opened pull request jekyll/jekyll-compose#24
@FellowMD
Reduce code duplictaion between commands
10 commits with 199 additions and 96 deletions
@FellowMD
  • @FellowMD b9387fe
    Reduce duplication of path & filetype calculations for page, post, draft
FellowMD commented on issue jekyll/jekyll-compose#21
@FellowMD

I think moving back to drafts makes more sense. Makes it the proper inverse of the publish command. There could be a retract command or something s…

FellowMD commented on issue jekyll/jekyll-compose#22
@FellowMD

$ cd my-site/subdir $ jekyll post blah Invalid command. Use --help for more information I didn't think it would matter, but I've never really dealt…

@FellowMD
page generation should create directory path if necessary
@FellowMD
Drop leading ./ from messages & errors?
@FellowMD
0.3.0 release
@FellowMD
0.3.0 release
FellowMD commented on issue jekyll/jekyll-compose#20
@FellowMD

Sure

FellowMD commented on issue jekyll/jekyll-compose#20
@FellowMD

Note the history.md is a little inaccurate at the moment, since I added an 0.2.2 entry which was never released.

@FellowMD
0.3.0 release
@FellowMD
FellowMD deleted branch add-page-generator at FellowMD/jekyll-compose