Skip to content

Conversation

@willfore
Copy link

Updated package to use npm's webpack build function for building assets. Updated to the 0.7 version of Bootleg. Updated the readme to reflect the changes needed.

@brienw
Copy link
Contributor

brienw commented Oct 17, 2018

Thanks for the PR, @willfore! Can you put the Elixir version down to 1.6 to match bootleg's current minimum (we've kept it there to match distillery's min) and there's some tricky version matching required between OTP and Elixir right now, so you'll have to set prepare.sh to use:

export ERLANG_VERSION="20.3"
export ELIXIR_VERSION="v1.6.0"

i suspect that will fix the reason the build is failing

Thanks again for making this change!

@willfore
Copy link
Author

No problem. All done! :)

mix.exs Outdated
[
app: :bootleg_phoenix,
version: @version,
elixir: "~> 1.7",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, please bump this down to 1.6 as well!

@brienw
Copy link
Contributor

brienw commented Oct 17, 2018

This new CI failure is due to more recent versions of credo getting really tripped up over some of our internal meta programming in bootleg. once sec, i'll adjust the credo config for bootleg_phoenix

@brienw
Copy link
Contributor

brienw commented Oct 17, 2018

@willfore I'm unable to push to your branch (this could be a config error on my side, i'm on a new laptop that i haven't fully set up yet), so could you just change line 86 of .credo.exs from:
{Credo.Check.Readability.StringSigils},
to:
{Credo.Check.Readability.StringSigils, false},

@willfore
Copy link
Author

Sure thing, incoming.

@brienw
Copy link
Contributor

brienw commented Oct 18, 2018

@willfore I've adjusted some package dependencies versions and adjusted some of the testing infrastructure to support this. Can you please add the necessary files to the fixture test/fixtures/drunkin_phoenix_1_4 so that it has something to build. After that is set, I think the only remaining thing is to add a note to the README.md about using older version of bootleg_phoenix for use with Phoenix < 1.4

You can run the tests locally via MIX_ENV=test mix test and you will need to make sure you have Docker installed. Lemme know if you have questions or need any help!

@willfore
Copy link
Author

willfore commented Oct 19, 2018

I added all the needed files to match the changes. I did not update the README in regards to using with Phoenix < 1.4 as I wasn't sure about version numbers. Let me know if anything else needs to be done.

@brienw
Copy link
Contributor

brienw commented Oct 19, 2018

This is great, thank you! I'll update the README today

@willfore willfore closed this by deleting the head repository Jul 30, 2024
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.

2 participants