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

Develop 3.0 mergerebar3 #248

Merged
merged 19 commits into from
May 7, 2020
Merged

Develop 3.0 mergerebar3 #248

merged 19 commits into from
May 7, 2020

Conversation

martinsumner
Copy link
Contributor

There was some work to tidy up cuttlefish in Ted's rebar3 branch. This work was required by the bitcask develop-3.0 branch to run its bitcask schema check unit tests.

I've merged in here, but ignoring most of the rebar.config changes (as the rebar3 branch didn't escriptize as expected a cuttlefish file).

There is an update to a cuttlefish_rebar_plugin module - to make it rebar2 not rebar compatible - but this is not yet rebar3 compatible. I can't see anything using this module - so the lack of rebar3 support is hopefully not an issue.

@martincox
Copy link
Contributor

I think cuttlefish is being pulled in (when making a release) from the kyorai fork via the rebar3_cuttlefish plugin, so we may need to bring that inline and merge with this at some point.

@martinsumner
Copy link
Contributor Author

It is @martincox branch of rebar3_cuttlefish plugin being used at the moment. I have this fork/branch prepared instead (which I've kept up to date with yours).

https://github.com/martinsumner/rebar3_cuttlefish/tree/develop-3.0

Will PR this into the basho version today.

@martincox
Copy link
Contributor

Ahh sound then 👍

@martinsumner martinsumner marked this pull request as ready for review May 7, 2020 11:07
@martinsumner martinsumner merged commit 9273682 into develop-3.0 May 7, 2020
@martincox martincox deleted the develop-3.0-mergerebar3 branch January 25, 2022 15:41
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.

3 participants