Skip to content

Commit

Permalink
actions: add pre-commit
Browse files Browse the repository at this point in the history
  • Loading branch information
booxter committed May 24, 2024
1 parent 4cefc44 commit e25aa78
Show file tree
Hide file tree
Showing 6 changed files with 15 additions and 6 deletions.
2 changes: 1 addition & 1 deletion .flox/env/manifest.lock
Original file line number Diff line number Diff line change
Expand Up @@ -1205,4 +1205,4 @@
"nixpkgs"
]
}
}
}
10 changes: 10 additions & 0 deletions .github/workflows/pre-commit.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
name: pre-commit
on: [push, pull_request]

jobs:
pre-commit:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/setup-python@v3
- uses: pre-commit/[email protected]
1 change: 0 additions & 1 deletion .github/workflows/pytest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,4 +17,3 @@ jobs:
run: |
pip install pytest
pytest
2 changes: 1 addition & 1 deletion src/letsrolld/cmd/update.py
Original file line number Diff line number Diff line change
Expand Up @@ -257,7 +257,7 @@ def get_offer_id(offer):
.filter_by(film_id=db_obj.id, offer_id=get_offer_id(offer.technical_name))
.first()
)
if obj is not None:
if obj is not None:
obj.url = offer.url
else:
obj = models.FilmOffer(
Expand Down
4 changes: 2 additions & 2 deletions ui/scripts/movie.js
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ export default class Movie {
// TODO: change for the dynamic image cover later
const imageElem = `<img class="movie-image-cover" src=${this.cover_url} alt="Watch this space, cover is comming soon" />`;
const cover = this.createMovieElemHTML('figure', 'movie-image', imageElem);

// handle the case if trailer is not available
if (this.trailer_url) {
//the html character code is an arrow directed to the right
Expand Down Expand Up @@ -123,7 +123,7 @@ export default class Movie {
return container;
}

// expects countries object that has name and flag fields, class
// expects countries object that has name and flag fields, class
getFlags(countries, name) {
let flags = "";

Expand Down
2 changes: 1 addition & 1 deletion ui/scss/components/_flags.scss
Original file line number Diff line number Diff line change
Expand Up @@ -51,4 +51,4 @@
}
}
}
}
}

0 comments on commit e25aa78

Please sign in to comment.