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

fix(status): prevent exit due to api request error #43

Merged
merged 1 commit into from
Feb 12, 2025
Merged

Conversation

benelan
Copy link
Owner

@benelan benelan commented Feb 12, 2025

  • Prevent gh-fzf from exiting if errors occur when fetching status info
  • Cache status response for 10min
  • Remove redundant request to githubstatus.com

- Prevent gh-fzf from exiting if errors occur when fetching status info
- Cache status response for 10min
- Remove redundant request to githubstatus.com
@benelan benelan merged commit c2008d5 into main Feb 12, 2025
1 of 2 checks passed
@benelan benelan deleted the status-caching branch February 12, 2025 23:15
benelan added a commit that referenced this pull request Feb 13, 2025
* origin/main:
  ci: add prettierignore
  fix(status): prevent exit due to api request error (#43)
benelan added a commit that referenced this pull request Feb 16, 2025
* origin/main:
  feat: add environment variables to configure keybinds (#45)
  docs: cleanup readme, simplify default command (#44)
  ci: add prettierignore
  fix(status): prevent exit due to api request error (#43)
  ci: migrate to shared workflows (#42)
benelan pushed a commit that referenced this pull request Feb 17, 2025
🤖 I have created a release *beep* *boop*
---


## [0.15.0](v0.14.1...v0.15.0)
(2025-02-17)


### Features

* Add `search` command
([#33](#33))
([c39d3ad](c39d3ad))
* Add environment variables to configure keybinds
([#45](#45))
([41b44b0](41b44b0))
* Add global help key to view command's readme section
([#46](#46))
([a7a076e](a7a076e))
* **pr:** Make draft pull request numbers gray
([63da381](63da381))


### Bug Fixes

* **status:** Prevent exit due to api request error
([#43](#43))
([c2008d5](c2008d5))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
benelan added a commit that referenced this pull request Feb 17, 2025
* origin/main:
  docs(changelog): fix linting error
  docs(search): remove hard line wraps from readme section
  chore(main): release 0.15.0 (#41)
  feat: add `search` command (#33)
  feat: add global help key to view command's readme section (#46)
  ci: update projections with new workflow name
  feat: add environment variables to configure keybinds (#45)
  docs: cleanup readme, simplify default command (#44)
  ci: add prettierignore
  fix(status): prevent exit due to api request error (#43)
  ci: migrate to shared workflows (#42)
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