search

RandomDSdevel / Learning-CPlusPlus-By-Reading-PPP2E

Documentation of my experiences learning to program in C++ by reading Bjarne Stroustrup's 'Programming: Principles and Practice Using C++,' Second Edition.
0 stars 1 forks source link

Fix yml parsing error #2

Closed bkeepers closed 7 years ago

bkeepers commented 7 years ago

In #1 @RandomDSdevel said:

No wonder the changes I was making to config.yml weren't taking effect…

Actually, it looks like you're being bitten by this issue: https://github.com/probot/settings/issues/38

YAMLException: incomplete explicit mapping pair; a key node is missed at line 3, column 119:
     ... Bjarne Stroustrup's 'Programming: Principles and Practice Using  ...

Since the description has a colon in it, yaml breaks. This fixes it by surrounding the description in double quotes.

RandomDSdevel commented 7 years ago

@bkeepers: Ah, of course. (How silly of me not to think of a solution as simple as that…)

RandomDSdevel commented 7 years ago

@bkeepers: OK, all fixed. Thanks for taking the time to contribute to such an inconsequential GitHub repository as this one.

P. S.: I'm going to amend the commit, though, now that merging it made me realized it got rid of the double-spacing I prefer to have between colons and the text that follows them…(P. P. S.: …and between sentences, too….)

RandomDSdevel commented 7 years ago

@bkeepers: Just to let you know (even though you probably don't even care any more…,) I've finished amending your commit post-merge — I really should have just amended it on your branch before merging, shouldn't I? — to make .github/config.yml's formatting match my stylistic preferences again. Thanks again for helping to maintain Probot's Settings…well, I guess you could call it a daemon of sorts, right? Anyway, have a nice day!