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

Improve variable & function names #417

Open
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@katrinleinweber
Copy link
Collaborator

commented Apr 18, 2019

Just a bunch of renamings that hopefully make the purpose of each renamed element clearer.

Show resolved Hide resolved _episodes_rmd/02-func-R.Rmd Outdated

@katrinleinweber katrinleinweber force-pushed the improve-names branch from 7d86a37 to f528213 Apr 21, 2019

@katrinleinweber katrinleinweber force-pushed the improve-names branch 2 times, most recently from 26898a6 to 00fa44a May 7, 2019

@katrinleinweber katrinleinweber force-pushed the improve-names branch from 00fa44a to d4903a8 May 7, 2019

@katrinleinweber katrinleinweber requested a review from diyadas May 7, 2019

@katrinleinweber

This comment has been minimized.

Copy link
Collaborator Author

commented May 7, 2019

@diyadas: I found another occurrence of fence thanks to #424 (comment) and renamed outside to edges because that seems more meaningful for the 1st & last element of a vector ;-) Please re-review & merge if you approve.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.