Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rework readme, add interactive example #10

Merged
merged 1 commit into from
Nov 14, 2015

Conversation

rixmann
Copy link
Member

@rixmann rixmann commented Sep 10, 2015

No description provided.

@rixmann
Copy link
Member Author

rixmann commented Sep 18, 2015

rafael gave his feedback per mail... just storing here to have it at the right place:

Ich wollte ja eine Mail schreiben mit einer Beschreibung, wo ich mit der Doku nicht weiter kam.
Es lag wohl an Tetrapak, und ich habe gehört (von Sascha), dass ihr das eigentlich nicht mehr verwendet.
Da die Doku jetzt überarbeitet ist, finde ich es sinnvoller, mir die neue Version anzusehen und hier zu kommentieren:

Gleich im ersten Satz könnte man „some requirements“ präzisieren oder auf die Beschreibungen der Voraussetzungen verlinken.

Bei Installation … "required" wäre es cool, die aufgeführte software zu verlinken, so wie zB. bei tetrapak schon der Fall. Das erleichtert Noobs wie mir den Einstieg.

Das problem mit lager_transform, parse_transform besteht immer noch, aber das scheint daran zu liegen, dass ich das hier mit OSX mache. (falls das von Interesse ist, hier nochmal was bei mir ausgegeben wird:
$ rebar compile
==> regine (compile)
/Users/friesen/tp/dike/deps/regine/src/regine_server.erl:none: undefined parse transform 'lager_transform'
ERROR: compile failed while processing /Users/friesen/tp/dike/deps/regine: rebar_abort
wenn ich in rebar.config lager_transform, parse_transform tausche, geht das aber problemlos. (das ist nicht nur bei dike ein problem, sondern auch bei anderen Projekten)

"tetrapak test" macht bei mir nun gar nichts. Es wirft keine Fehler mehr, gibt aber sonst auch nichts aus.

installation und test mit mix funktioniert bei mir. (Test: 1 skipped, 5 ok)

"using dike" schaffe ich jetzt hier nicht mehr. wenn ich die Tage dazu komme, dann schreibe ich noch mal.

umatomba added a commit that referenced this pull request Nov 14, 2015
rework readme, add interactive example
@umatomba umatomba merged commit 0156476 into travelping:master Nov 14, 2015
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.

None yet

2 participants