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
Switch to contiguous idl #46
Comments
See #41 :) |
What's the status of #41, can we merge it, @dontcallmedom , @martinthomson ? |
since you're an editor now, you can probably make that decision :) That being said, I think the PR would need updating given the commits that were merged since it was generated. If you could review the PR in its previous state, given thumb up to the overall approach, I can then look into the best ways to update it (regenerate from source vs manual cherry-pick) |
I realize that you didn't have write access to the repo - sorry about that, should be fixed now. |
@dontcallmedom, I took a look at the #41 PR and it looks fine, just throw away references to |
Ominous warning in the ReSpec means that this Spec should be transformed to the new thing:
@dontcallmedom, after doing this transition by hand elsewhere I'm more than willing to be shown your conversion script in action :-)
The text was updated successfully, but these errors were encountered: