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

Rename scripts directory to ci #37

Merged
merged 1 commit into from
Jan 12, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions .github/workflows/testing.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: Testing
name: Linting and Testing

on:
push:
Expand All @@ -23,7 +23,7 @@ jobs:
- name: Cache dependencies
uses: Swatinem/[email protected]
- name: Prep data and sqlx # This prevents sqlx errors if the database isn't running
run: sh ./scripts/prep_tests.sh
run: sh ./ci/prep_tests.sh
- name: Run cargo clippy
run: cargo clippy --all-targets -- --deny warnings
fmt:
Expand All @@ -49,6 +49,6 @@ jobs:
- name: Cache dependencies
uses: Swatinem/[email protected]
- name: Prep data and sqlx # This prevents sqlx errors if the database isn't running
run: sh ./scripts/prep_tests.sh
run: sh ./ci/prep_tests.sh
- name: Run cargo test
run: cargo test
File renamed without changes.
File renamed without changes.
5 changes: 5 additions & 0 deletions ci/prep_tests.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
#!bin/bash

mkdir ci/data
cp pieces/postgres_etl/data/persons.csv ci/data
docker compose -f ci/docker-compose.yml up -d
5 changes: 0 additions & 5 deletions scripts/prep_tests.sh

This file was deleted.