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

chore: split Elixir tasks #375

Merged
merged 3 commits into from
Jun 28, 2024
Merged

Conversation

hauleth
Copy link
Contributor

@hauleth hauleth commented Jun 27, 2024

What kind of change does this PR introduce?

Split Elixir tasks in CI to improve build times.

@hauleth hauleth requested a review from a team as a code owner June 27, 2024 21:34
@hauleth hauleth force-pushed the chore/split-elixir-tasks branch 19 times, most recently from 5f118de to db2dd63 Compare June 28, 2024 08:50
.github/workflows/elixir.yml Outdated Show resolved Hide resolved
mix.lock Outdated Show resolved Hide resolved
@hauleth hauleth force-pushed the chore/split-elixir-tasks branch 3 times, most recently from 15842f7 to 0d11917 Compare June 28, 2024 09:33
.github/workflows/prod.yml Outdated Show resolved Hide resolved
.github/workflows/stage.yml Outdated Show resolved Hide resolved
@hauleth hauleth force-pushed the chore/split-elixir-tasks branch 2 times, most recently from ac2108b to 86b49ff Compare June 28, 2024 09:42
@hauleth hauleth force-pushed the chore/split-elixir-tasks branch 2 times, most recently from f30a25c to 0abf0b7 Compare June 28, 2024 09:46
.github/workflows/prod.yml Outdated Show resolved Hide resolved
- Use `erlef/setup-beam` action for setting up the tools instead of
  hand-crafted script
- Reorder steps in the CI to run lints faster to improve feedback loop
@hauleth hauleth force-pushed the chore/split-elixir-tasks branch 3 times, most recently from 4891ec4 to 74f266f Compare June 28, 2024 10:43
@hauleth
Copy link
Contributor Author

hauleth commented Jun 28, 2024

Some problems with Dialyzer when ran in test environment - https://github.com/supabase/supavisor/actions/runs/9711545084/job/26804706396?pr=375#step:10:31

@abc3
Copy link
Member

abc3 commented Jun 28, 2024

we can ignore it via .dialyzer_ignore.exs

@hauleth hauleth force-pushed the chore/split-elixir-tasks branch 5 times, most recently from 698f776 to 2daf6b2 Compare June 28, 2024 13:05
Split CI tasks related to Elixir to improve build times by parallelizing
different steps.
@hauleth hauleth requested a review from abc3 June 28, 2024 13:19
Copy link
Member

@abc3 abc3 left a comment

Choose a reason for hiding this comment

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

🔥 🔥 🔥

@abc3 abc3 merged commit 9f56b87 into supabase:main Jun 28, 2024
7 checks passed
@hauleth hauleth deleted the chore/split-elixir-tasks branch July 8, 2024 14:46
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