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

Publish to ForgeBox.io? #56

Open
jfrux opened this issue Feb 9, 2017 · 3 comments
Open

Publish to ForgeBox.io? #56

jfrux opened this issue Feb 9, 2017 · 3 comments

Comments

@jfrux
Copy link
Contributor

jfrux commented Feb 9, 2017

I noticed this had a box.json but isn't published to forgebox.io.
Would someone be able to do this?
Or would you mind if I do it?

@russplaysguitar
Copy link
Owner

I'd be more comfortable with that if we got the build passing. As it is, I'm concerned about stability. #54 almost fixes it, albeit in a way I'm less happy about (relying on Java).

@jfrux
Copy link
Contributor Author

jfrux commented Feb 10, 2017 via email

@russplaysguitar
Copy link
Owner

None of the other methods require Java as far as I recall. I intentionally tried not to use Java when creating the library so as to avoid dependency issues (like the ones we're seeing in the failing tests now).

The problem is that I'm not sure there's another way to resolve the issue (easily). The logic (ported from JS) expects the arguments struct to be sorted, but this is not really possible in CF structs. Not sure why this wasn't a problem in CF 10.

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

No branches or pull requests

2 participants