search

greplytix / Hamlbars

Sublime Text 3 Syntax highlighting for .hamlbars files which is a combination of both Haml and Handlebars
MIT License
2 stars 0 forks source link

Push general practice with Python to a repo #1

Closed ghost closed 9 years ago

ghost commented 9 years ago

Completion of this task will also imply quintessential practice with Sublime Text 3 and Git via a shell.

All student members of @greplytix/siuc should update their respective status on this issue up to and including completion of the task.

The members of said team have been invited to a collaborate on a private repo for practice.

ghost commented 9 years ago

I pushed a tail demo written in Python to said repo. It demonstrates my knowledge of Python-specific concepts, such as generators utilizing the 'yield' keyword, and file I/O utilizing the 'with' keyword. My implementation is extremely efficient in terms of memory, trumping library functions like [file object].readlines().

The time complexity, at worst, is a bit higher than I would prefer, but that is in order to keep track of position of the utilized file. This way, one could read the last ten lines, then call the same function again to read the lines that were above the last ten, and so on, all without eating up hardly any memory. I owe many thanks to the implementation of generator functions to this end.

On average, however, running tail for a limited number of lines on the usual text or log file is just a temporary endeavor that would negligibly detract from the flow of processes on a given system. Thus, all in all I'm quite pleased with the efficiency of my demo.

anthonykulis commented 9 years ago

@reesington not 100% what this has to do with the hamlbars plugin. Lets keep an eye on which repo is which. Thanks for jumping right in though.

anthonykulis commented 9 years ago

Nevermind I understand what you were doing now. Lets keep the commit comments concise. You need to merge your work into master. You do this via a "pull request". Pull that and close this thread. I will create two tickets for @jonlegacy and moe. He currently is not a member yet. Will be shortly.

anthonykulis commented 9 years ago

Oh, be sure to close your ticket when done. Also, review github commit messages. You can use things like "#1" in your commit message to link this ticket to the commit. So please include that.

ghost commented 9 years ago

Sounds good, Anthony.

Before I make any assumptions, please clarify something for me: You want me to merge my tail demo from the private repo with the Hamlbars master? Is that correct?

Also, thanks for the note on commit messages. I did not know that, but now I do. I will employ that for this and future issues.

anthonykulis commented 9 years ago

talking about commit a379fa89c01e1356e10fc461615c6588aef72e5b on your reese branch. We will cover it today.