Include dynamic helpers #40

Closed
sjaeckel opened this Issue Feb 26, 2014 · 4 comments

Projects

None yet

3 participants

@sjaeckel
Member

@buggywhip proposed some internal helper functions, to ease integration of ltc in other languages. They should be reviewed and merged

@sjaeckel sjaeckel added this to the v1.18.0 milestone Feb 26, 2014
@buggywhip

On Feb 26, 2014, at 12:20 AM, Steffen Jaeckel notifications@github.com wrote:

@buggywhip proposed some internal helper functions, to ease integration of ltc in other languages. They should be reviewed and merged

Steffen,

I have some refinements I want to post. The other day I learned I have read only access so my commits won't post to the server. Earlier today I tried to make a new branch "dynHlp2" but got side tracked almost immediately. Soooo...

Can you grant me write/commit access to "dynHlp"? ...or shall I go with plan B and create the fresh branch "dynHlp2" Either should allow me to post my changes.

Larry

@sjaeckel
Member

I gave you write access to the repository.

You can either continue with the branch, or you rebase it on develop and force-push it.

Steffen

@rofl0r
rofl0r commented Feb 26, 2014

@buggywhip why dont you just click on "fork", do your work in your fork, and then click on "pull request" once you're done like anyone else does ? i dont think its a good idea if other ppl start force pushing unreviewed stuff to develop at this point.

@sjaeckel
Member
sjaeckel commented Mar 7, 2014

#41 handles this

@sjaeckel sjaeckel closed this Mar 7, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment