belaustegui
I’ve noticed that Phoenix has shifted from using Earthly for running the entire CI tests to using it only for the integration tests, with the rest of the CI jobs now running in standard GitHub CI workflows.
This PR did the change: Reserve earthly for integration only by josevalim · Pull Request #4964 · phoenixframework/phoenix · GitHub
I’m interested to know why this change was made but couldn’t find any information. Was it because of performance improvements? Better maintainability? Something else?
Where Next?
Trending in Discussions
Other Trending Topics
Latest Phoenix Threads
Chat & Discussions>Discussions
Latest on Elixir Forum
Sponsor Spotlight
Producing high quality Elixir screencasts since 2017.
Categories:
Sub Categories:
Forums
Popular Tags
- #ecto
- #liveview
- #troubleshooting
- #learning-elixir
- #deployment
- #library
- #erlang
- #testing
- #genserver
- #mix
- #absinthe
- #remote-other
- #otp
- #plug
- #how-to-question
- #macros
- #postgres
- #channels
- #elixirconf
- #exunit
- #discussion
- #code-sync
- #javascript
- #podcasts
- #onsite
- #dialyzer
- #docker
- #authentication
- #umbrella
- #full-time-contract
- #podcasts-by-brainlid
- #ecto-query
- #elixir-ls
- #blog-post
- #phoenix_html
- #iex
- #graphql
- #ai
- #genstage
- #elixirconf-us
- #websockets
- #supervisor
- #advent-of-code
- #distillery
- #processes
- #api
- #forms
- #metaprogramming
- #security
- #hex
Our Sponsors
Build Elixir applications with speed and confidence.
Supporting innovation across the BEAM ecosystem.
We build reliable cloud platforms for business-critical systems.
Catch errors, track performance, monitor hosts and more.
Error tracking, logs, uptime, and Just Enough APM. Built for developers.
Practical resources that improve the lives of professional developers.
Develop your skills with books, videos, and courses.
Enabling companies to succeed by building software people love.
The team behind Membrane, Popcorn, LiveDebugger. Available for hire.
Courses that'll move you from confusion to "Aha, now I get it!"
Showing Posts 1 to 2- Show Best Posts
- Show All Posts (oldest first)
- Show All Posts (newest first)
josevalim
I personally find Earthly great when you have external dependencies (such as a database), which is not the case for the regular
mix test. So I thought it makes sense to not require it there.belaustegui
Oh of course! That’s a ver reasonable change.
I asked because we are using Earthly as well and I was surprised to see it go at least for part of the tests. Didn’t notice that those tests don’t require any external dependency, which makes perfect sense.
Thanks!