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

GitHub PAT usage, coverage bump #10

Merged
merged 3 commits into from
May 18, 2023
Merged

GitHub PAT usage, coverage bump #10

merged 3 commits into from
May 18, 2023

Conversation

azemetre
Copy link
Collaborator

No description provided.

azemetre added 3 commits May 18, 2023 17:45
Enabling switch statement for URL building (self, user, org)

GitHub authorization enabled through use of personal access token (PAT).
If token is empty don't pass token to headers.

Using a Sleep interface to make testing less painful (and faster).

More appropriate code comments for functions

tests: adding coverage to all three piscator functions

Decided to opt out of testing file systems for easier code complexity
chore: renaming to something more fishy
@azemetre azemetre added enhancement New feature or request build ci/cd, linting, formatting, release management tests anything involving testing, suites of tests, types of tests, structuring tests labels May 18, 2023
@azemetre azemetre self-assigned this May 18, 2023
@azemetre azemetre merged commit ed01c0e into main May 18, 2023
@azemetre azemetre deleted the github-auth-test-coverage branch May 18, 2023 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build ci/cd, linting, formatting, release management enhancement New feature or request tests anything involving testing, suites of tests, types of tests, structuring tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant