Skip to content

Comments

Rewrite for v3.0.0#6

Draft
Wizofgoz wants to merge 21 commits intomasterfrom
Rewrite
Draft

Rewrite for v3.0.0#6
Wizofgoz wants to merge 21 commits intomasterfrom
Rewrite

Conversation

@Wizofgoz
Copy link
Member

New Features

  • Request Builders
  • Fully objectified requests/responses
  • Iterable paginated responses
  • Automatic retry
  • Constants for Filter Operators, Sort Directions, and Date Format
  • Automated tests
  • Removed vendor directory from source control
  • Upped PHP requirement to 7.2
  • GitHub Actions workflow
  • Automated linting and static analysis tools (PHP-CS-Fixer, Phan)

@Wizofgoz Wizofgoz requested a review from tolson17 September 17, 2020 01:25
@codecov
Copy link

codecov bot commented Sep 18, 2020

Codecov Report

❗ No coverage uploaded for pull request base (master@2f4d604). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master       #6   +/-   ##
=========================================
  Coverage          ?   57.14%           
  Complexity        ?      532           
=========================================
  Files             ?       47           
  Lines             ?     1085           
  Branches          ?        0           
=========================================
  Hits              ?      620           
  Misses            ?      465           
  Partials          ?        0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2f4d604...265fbcc. Read the comment docs.

@Wizofgoz Wizofgoz marked this pull request as draft September 30, 2020 21:18
@Wizofgoz
Copy link
Member Author

@tolson17 I flagged this as a draft to add more tests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant