Skip to content

refactor: improve GitHub API interactions and error handling #19

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

Conversation

virgofx
Copy link
Member

@virgofx virgofx commented Oct 14, 2024

This pull request refactors the GitHub API interactions and error handling in the codebase. The changes include:

  • Separating the GitHub API interactions from the shared module github.ts and into the purpose-specific file (e.g. tags, releases).
  • Improving error handling for GitHub API requests.
  • Adding more detailed logging for GitHub API requests.
  • Dropping @actions/github and instantiating GitHub context and Octokit directly. This allows us to leverage proper instanceof RequestError.
  • Updating the README.md file to reflect the changes.

Fixes #17

@virgofx virgofx force-pushed the 17-refactor-github-workflow-to-separate-context-and-input-configuration branch 30 times, most recently from 9dd0138 to 5e68751 Compare October 14, 2024 18:22
@virgofx virgofx force-pushed the 17-refactor-github-workflow-to-separate-context-and-input-configuration branch 20 times, most recently from c572917 to 70e4cd6 Compare October 14, 2024 23:36
@virgofx virgofx changed the title misc refactor: improve GitHub API interactions and error handling Oct 14, 2024
@virgofx virgofx force-pushed the 17-refactor-github-workflow-to-separate-context-and-input-configuration branch from 70e4cd6 to c036117 Compare October 14, 2024 23:38
This commit refactors the GitHub API interactions and error
handling in the codebase. The changes include:

- Separating the GitHub API interactions from the shared module `github.ts`
  and into the purpose-specific file (e.g. tags, releases).
- Improving error handling for GitHub API requests.
- Adding more detailed logging for GitHub API requests.
- Drop @actions/github and instantiate GitHub context and Octokit
  directly. This allows us to leverage proper instanceof RequestError.
- Updating the README.md file to reflect the changes.

Fixes #17
@virgofx virgofx force-pushed the 17-refactor-github-workflow-to-separate-context-and-input-configuration branch from c036117 to e32668f Compare October 14, 2024 23:45
Copy link

Release Plan

No terraform modules updated in this pull request.

✅ Wiki Check

Note: The following Terraform modules no longer exist in source; howevever corresponding tags/releases exist. Automation tag/release deletion is disabled and no subsequent action will take place.

  • v1.0.2
  • v1.0.1
  • v1.0.0
  • v1

@virgofx virgofx marked this pull request as ready for review October 14, 2024 23:48
@virgofx virgofx merged commit 1f1370b into main Oct 14, 2024
21 checks passed
@virgofx virgofx deleted the 17-refactor-github-workflow-to-separate-context-and-input-configuration branch October 14, 2024 23:48
@virgofx virgofx added the enhancement New feature or request label Oct 15, 2024
@virgofx virgofx self-assigned this Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refactor GitHub Workflow to Separate Context and Input Configuration
1 participant