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

Change ./checkin.py into an entrypoint #71

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 1 commit
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
Prev Previous commit
Next Next commit
fixes travis command and updates README instructions
  • Loading branch information
lockefox committed Aug 18, 2021
commit 80451c511388846e70c8f96393f811b70b919fa0
2 changes: 2 additions & 0 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@ language: python
env:
global:
- CC_TEST_REPORTER_ID=3d0abe23b60c405d5de7e038f366649a8002c4ccedccabad3da0e3b3e9b5049b
install:
pip install .[dev]
python:
- "2.7.15"
- "3.7.1"
Expand Down
4 changes: 0 additions & 4 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -11,10 +11,6 @@ lint:
docker:
docker build -t pyro2927/southwestcheckin .

docker-test: docker
docker run --rm -it pyro2927/southwestcheckin bash


release:
docker push pyro2927/southwestcheckin

Expand Down
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -35,13 +35,13 @@ This script can either be ran directly on your host or within Docker.
#### Install Base Package Requirements

```bash
$ pip install -r requirements.txt
$ pip install .
```

#### Usage

```bash
$ python ./checkin.py CONFIRMATION_NUMBER FIRST_NAME LAST_NAME
$ checkin CONFIRMATION_NUMBER FIRST_NAME LAST_NAME
```

### Docker
Expand Down