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

Create django.yml #29

Open
wants to merge 44 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
44 commits
Select commit Hold shift + click to select a range
dfae2ed
Create django.yml
sindrelothe Mar 29, 2022
220f410
Change files to correct project in pylint command
sindrelothe Mar 29, 2022
1fa6248
change branch name to "main"
sindrelothe Mar 29, 2022
ee538f9
Merge branch 'main' into sindrelothe-patch-1
sindrelothe Apr 21, 2022
2b339e4
fix linting errors for some files
sindrelothe Apr 21, 2022
1835c9b
fix linting errors for choices.py, /fields/__init__, weekday.py
sindrelothe Apr 21, 2022
a4dabe5
Change pylint disable codes to names
sindrelothe Apr 26, 2022
ef09acb
Fix more pylint errors
sindrelothe Apr 26, 2022
6984709
Remove eval function from heroku.py
sindrelothe Apr 26, 2022
d68104b
fix final pylint errors
sindrelothe Apr 26, 2022
6b928a8
fix final pylint error
sindrelothe Apr 26, 2022
32ac51d
merge with main
sindrelothe Apr 26, 2022
79f7de5
remove help_text for Tag class
sindrelothe Apr 26, 2022
b468c80
add trailing comma for lists in tag.py
sindrelothe Apr 26, 2022
b0bcc49
added trailing comma to urls.py
sindrelothe Apr 26, 2022
b6f33be
Add specific exception to apps.py
sindrelothe Apr 26, 2022
bb0ddfb
disable fixme from pylint
sindrelothe Apr 26, 2022
443fd8d
fix migrate bug
sindrelothe Apr 26, 2022
0811664
fix error with github tests
sindrelothe Apr 26, 2022
13a4756
fix error with github tests
sindrelothe Apr 26, 2022
bd767eb
copy fix from rekenett for github tests
sindrelothe Apr 26, 2022
351a541
run migrate before pylint in django.yml'
sindrelothe Apr 26, 2022
576cad0
add settings config to apps.py
sindrelothe Apr 26, 2022
5dfa0d6
remove introduced error in apps.py
sindrelothe Apr 26, 2022
f64388d
add DJANGO_SETTINGS_MODULE to yml script
sindrelothe Apr 26, 2022
78b200c
Update django.yml
sindrelothe Apr 26, 2022
e0581c1
Update django.yml
sindrelothe Apr 26, 2022
2cf9ff5
add new line
sindrelothe Aug 23, 2022
ba0bcd2
fix pipeline
sindrelothe Aug 23, 2022
7df6683
fix pipeline
sindrelothe Aug 23, 2022
9111c6d
move verify migrations to before run pylint in yml
sindrelothe Aug 23, 2022
4946d49
add makemigrations to yml
sindrelothe Aug 23, 2022
227240a
merge main
sindrelothe Sep 27, 2022
31c5fe4
fix merge conflicts
sindrelothe Sep 27, 2022
e859ea0
fix merge conflicts
sindrelothe Sep 27, 2022
ba5254f
update python version in django.yml
sindrelothe Sep 27, 2022
4d1d07e
remove arguments from 'makemigrations' in djano.yml
sindrelothe Sep 27, 2022
a1fba86
remove migrations file
sindrelothe Sep 27, 2022
a0588f9
remove pylint from django.yml
sindrelothe Sep 27, 2022
095ba29
remove run pytest from infoscreen
sindrelothe Sep 27, 2022
b018e31
add linting to django.yml
sindrelothe Sep 27, 2022
ab1d4b3
remove linting of root
sindrelothe Sep 27, 2022
556722f
remove linting of root
sindrelothe Sep 27, 2022
0c2bd5f
use .sh structure to lint
sindrelothe Sep 27, 2022
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
2 changes: 1 addition & 1 deletion .github/workflows/django.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,4 +46,4 @@ jobs:
python -m pipenv run python manage.py migrate

- name: Run pylint
run: python -m pipenv run pylint infoscreen
run: python -m pipenv run pylint ./run-pylint.sh
Copy link
Member

Choose a reason for hiding this comment

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

Hmm er dette riktig?

2 changes: 1 addition & 1 deletion run-pylint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@
PROJECT_DIR=$(pwd)
PYTHONPATH="$PROJECT_DIR:$PYTHONPATH"
echo "PYTHONPATH=$PYTHONPATH"
PYTHONPATH=$PYTHONPATH pylint infoscreen
PYTHONPATH=$PYTHONPATH pylint infoscreen root
Copy link
Member

Choose a reason for hiding this comment

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

Ja det ser riktig ut.