Navigation Menu

Skip to content
This repository has been archived by the owner on Jun 20, 2022. It is now read-only.

edn has moved #3

Merged
merged 4 commits into from May 22, 2013
Merged

edn has moved #3

merged 4 commits into from May 22, 2013

Conversation

lenards
Copy link
Contributor

@lenards lenards commented May 22, 2013

I know this is a work-in-progress, but I just thought I'd do a silly grunt edit to help out.

Added a slightly nasty up-navigation path traversal, because the relative link wasn't going to Rich Hickey page, it was going to:

https://github.com/seancribbs/edn-erlang/blob/master/richhickey/edn

@seancribbs
Copy link
Owner

Is there a reason you can't use an absolute path? Does /edn-format/edn or //github.com/edn-format/edn not work?

@lenards
Copy link
Contributor Author

lenards commented May 22, 2013

Nope - it could just be https://github.com/edn-format/edn or //github.com/* works.

@seancribbs
Copy link
Owner

Thanks, merging. JUST FYI this is not a fully functional implementation yet.

seancribbs added a commit that referenced this pull request May 22, 2013
@seancribbs seancribbs merged commit 5abe0df into seancribbs:master May 22, 2013
@lenards
Copy link
Contributor Author

lenards commented May 22, 2013

I figured with the (WIP) in the title that it was on-going. I just wanted to pitch in with the little hyperlink edit. Thanks for riak_id too (using it now to learn about building riak_core apps).

@seancribbs
Copy link
Owner

@lenards You're very welcome. Thanks again for the PR!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants