1. Nathan Durnan
  2. TimeStampMod
  3. Pull requests

Pull requests

#4 Merged
Repository
domainfun
Branch
default
Repository
nedmech
Branch
default

Mostly cosmetic changes (except commit 0f1d032), mainly PEP8 compliance and code comment typos

Author
  1. domainfun
Reviewers
Description

I'm not sure whether I should do this Pull-Request as well, or whether you can pull anyway - if you're OK with my changes, that is.

Also, I'm not sure whether I should have put my commits on a new branch rather than on trunk.

Many thanks,

H.

  • Learn about pull requests

Comments (2)

  1. Nathan Durnan repo owner

    Excellent! For some reason, I thought I had already made everything use spaces instead of tabs, but apparently not. And great job resolving my typos. I will attempt to pull this, although I'm hoping to be able to pull it to the "development" branch instead of the "default" branch.

    For future changes, please work from the "development" branch to keep the workflow consistent. I try to keep the "default" branch as my clean branch for releases.

    I really appreciate your input and the work you've contributed. Thank you!

  2. Nathan Durnan repo owner

    I pulled your changes, but then rebased them to a new branch off of "development" and stripped them from "default". This will probably mess up your fork, so I apologize for any problems that may create.