Skip to content

Add highlighting of known_hosts files#6

Merged
lalyos merged 12 commits intolalyos:masterfrom
walles:walles/known-hosts
Jun 19, 2017
Merged

Add highlighting of known_hosts files#6
lalyos merged 12 commits intolalyos:masterfrom
walles:walles/known-hosts

Conversation

@walles
Copy link
Copy Markdown
Contributor

@walles walles commented Apr 20, 2017

Here's what it looks like with the Atom Light syntax theme:
highlighted-known-hosts

I get to edit those from time to time and getting this highlighting in here would make that experience just a little bit better :).

@walles
Copy link
Copy Markdown
Contributor Author

walles commented Jun 19, 2017

@lalyos would you like me to squash my changes into one?

@lalyos lalyos merged commit fc9ceb8 into lalyos:master Jun 19, 2017
@walles walles deleted the walles/known-hosts branch June 19, 2017 14:55
@walles
Copy link
Copy Markdown
Contributor Author

walles commented Jun 19, 2017

Thanks for merging @lalyos!

Releasing is pretty easy, so if you could make a release right now with only this change added that would be awesome!

Then, if the other PR gets some unit tests, just make another one after merging that.

To paraphrase Mr. Balmer: "Delivery, delivery, delivery!" :) /J

@walles
Copy link
Copy Markdown
Contributor Author

walles commented Jul 6, 2017

@lalyos Could you please apm publish minor?

I'd love to be able to use my changes!

@walles
Copy link
Copy Markdown
Contributor Author

walles commented Aug 24, 2017

@lalyos, could you please make a new release now that this change has been merged?

@lalyos
Copy link
Copy Markdown
Owner

lalyos commented Aug 25, 2017

done:

Preparing and tagging a new version ✓
Pushing v0.12.0 tag ✓
Publishing ssh-config@v0.12.0 ✓

Sorry, it took me soo long, but im on vacation, and moved to a new laptop, and lost all kinds of tokens/access ...

@walles
Copy link
Copy Markdown
Contributor Author

walles commented Aug 26, 2017

Thanks for releasing @lalyos, just tested it and your new release works like a charm :).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants