Skip to content

feat: improve error verbosity and resilience#4

Merged
kwaszczuk merged 4 commits intomainfrom
kwaszczuk/patch-2
Apr 3, 2025
Merged

feat: improve error verbosity and resilience#4
kwaszczuk merged 4 commits intomainfrom
kwaszczuk/patch-2

Conversation

@kwaszczuk
Copy link
Copy Markdown

@kwaszczuk kwaszczuk commented Apr 2, 2025

Motivation

Recently Jira API become a little unstable, randomly returning 403 for valid requests. To workaround this issue I added retries to those requests using axios-retry library.

Additionally, I improved the logging verbosity when any error occurs to make it easier to troubleshoot an issue once it occurs.

Last but not least, I applied some linting changes made by biome.

Quality Assurance

Tested by triggering the Kanar version from this PR here: https://github.com/RampNetwork/ramp-instant/pull/23464

Copilot AI review requested due to automatic review settings April 2, 2025 14:47
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enhances runtime error visibility by improving error logging and adjusting messaging behavior during task checks.

  • Updated string quote style to double quotes for consistency.
  • Enhanced error logging in jiraRequest to include additional error details.
  • Adjusted checkTasks messaging and ensured an explicit process exit when no tasks are found.
Files not reviewed (1)
  • package.json: Language not supported

Comment thread dangerfile.js
Comment thread dangerfile.js Outdated
@kwaszczuk kwaszczuk force-pushed the kwaszczuk/patch-2 branch 2 times, most recently from b245af7 to 85ab5bf Compare April 2, 2025 15:28
@kwaszczuk kwaszczuk force-pushed the kwaszczuk/patch-2 branch from 82546f8 to ac15bf8 Compare April 2, 2025 15:38
@kwaszczuk kwaszczuk force-pushed the kwaszczuk/patch-2 branch 2 times, most recently from 35796c3 to 270d066 Compare April 2, 2025 15:58
@kwaszczuk kwaszczuk force-pushed the kwaszczuk/patch-2 branch from 270d066 to 2e4ee64 Compare April 2, 2025 16:10
@kwaszczuk kwaszczuk changed the title feat: enhance runtime error visibility feat: make Kanar more error-resilient and improve error verbosity Apr 3, 2025
@kwaszczuk kwaszczuk changed the title feat: make Kanar more error-resilient and improve error verbosity feat: improve error verbosity and resilience Apr 3, 2025
Comment thread action.yml
@kwaszczuk kwaszczuk merged commit e9cb9eb into main Apr 3, 2025
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.

3 participants