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.

Sign up for GitHub

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

Jump to bottom

Release alpha 0.1 #3

Merged
merged 47 commits into from
May 1, 2018
Merged

Release alpha 0.1 #3

merged 47 commits into from
May 1, 2018

Conversation

Kilo59
Copy link
Owner

@Kilo59 Kilo59 commented Apr 28, 2018

No description provided.

@Kilo59
Create README.md
ca16695
@Kilo59
Setup repo structure
6ebca55
@Kilo59
Add context, tests
c8aa830
@Kilo59
add CI files
d6b086d
@Kilo59
add Travis build badge
7d9a907
@Kilo59
script stub with Workbook Template
415167c
@Kilo59
Fix Tests
f3f9574
@Kilo59
fix build
adf3922
@Kilo59
ignore
71019c3
@Kilo59
add fixpath() to context
37f614a
@Kilo59
upgrade pip to fix build issue on 3.7dev
5317a1a
@Kilo59
exclude 3.7-dev add code_coverage
9681651
@Kilo59
add CodeClimate Test id
5edde05
@Kilo59
add badges
4a89484
@Kilo59
Merge remote-tracking branch 'refs/remotes/origin/master' into dev
a95aa74
@Kilo59
pull credential values from Setup WS
2564ec3
@Kilo59
flake8 corrections
43af4a5
@Kilo59
pep8, fixpath
146b1f3
@Kilo59
rename functions, add get_picklist() function stub
5dd66a5
@Kilo59
ignore .pdf .PNG
62b0933
@Kilo59
add clear_pycache.bat
28b354f
@Kilo59
pep8 fix
54f491b
@Kilo59
ignore pytest cache
5904558
@Kilo59
Add Authorizer class
112ca65
@Kilo59
add verbose flag
105717a
@Kilo59
add check for API_key
53bb3f2
@Kilo59
Add basic tests for Authorizer
b6dac65
@Kilo59
Authorize and pull expense codes
9eb4051
@Kilo59
working expenser (rough draft)
c6f3e2f
@Kilo59
cleanup and remove uneeded functions
ca752eb
@coveralls
Copy link

Pull Request Test Coverage Report for Build 28

  • 29 of 60 (48.33%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-14.4%) to 51.19%
Changes Missing Coverage Covered Lines Changed/Added Lines %
bt_expense/bt_expense.py 29 60 48.33%
Totals Coverage Status
Change from base Build 12: -14.4%
Covered Lines: 43
Relevant Lines: 84
💛 - Coveralls

1 similar comment
@coveralls
Copy link

Pull Request Test Coverage Report for Build 28

  • 29 of 60 (48.33%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-14.4%) to 51.19%
Changes Missing Coverage Covered Lines Changed/Added Lines %
bt_expense/bt_expense.py 29 60 48.33%
Totals Coverage Status
Change from base Build 12: -14.4%
Covered Lines: 43
Relevant Lines: 84
💛 - Coveralls

@coveralls
Copy link

coveralls commented Apr 28, 2018 •
edited
Loading

Coverage Status

Coverage decreased (-22.2%) to 43.396% when pulling 4a1bd74 on feature/alpha_0.1 into 7f22a3f on master.

@Kilo59
only submit entries that have a cost and date value set
b079ce2 
Only pull the first 100 rows of data
@Kilo59
split post_expenses() into 2 seperate functions
4e28335 
`prep_expenses()` returns a list of expense dictionaries.
`post_expenses()` iterates through a list of expense dictionaries and posts them.
str-eat
str-eat approved these changes May 1, 2018
View reviewed changes
Copy link
Collaborator

@str-eat str-eat left a comment

Choose a reason for hiding this comment

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

Looks good

@str-eat str-eat merged commit 6a09c3e into master May 1, 2018
@Kilo59 Kilo59 deleted the feature/alpha_0.1 branch May 19, 2018 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@str-eat str-eat str-eat approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@Kilo59 @coveralls @str-eat