RocHack / bb

Command line Blackboard client
MIT License
72 stars 8 forks source link

bb_courses improvements #19

Closed joeljk13 closed 9 years ago

joeljk13 commented 9 years ago

Format Improvement

Since the stdout from bb_courses is only shown the user, I thought it'd be good to have a more readable format. (As a side bonus, I think this new format will also be easier to read with awk.)

Bug Fix

The bug is that courses that begins with a number have that number removed (e.g. "20TH CENTURY PHYSICS" shows as "TH CENTURY PHYSICS". I just did a simple regex change to fix it; I don't think it'll mess up anyone else's courses, but I'd rather not assume that with Blackboard.

clehner commented 9 years ago

Good catch with the numbers beginning titles.

I like the new format. It is easier to read.

joeljk13 commented 9 years ago

Wow, I though \\+ was universal. Anyway, I've updated it to s/^[A-Z]*/& /, which also works with my sed (GNU 4.2.2).

clehner commented 9 years ago

Great, merging.