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.

Sign up for GitHub

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

Jump to bottom

fix: ignore stderr if qmate is not executed from a valid git repo #337

Merged
merged 8 commits into from
Jan 30, 2025
Merged

fix: ignore stderr if qmate is not executed from a valid git repo #337

merged 8 commits into from
Jan 30, 2025

Conversation

DIFSRIP
Copy link
Collaborator

@DIFSRIP DIFSRIP commented Jan 29, 2025 •
edited
Loading

  • Fixes an issue where the repo information for that stats tracker could not be gathered when qmate is called from an absolute path
  • Hides internal errors from stderr when qmate is executed from non-repository location

@DIFSRIP DIFSRIP marked this pull request as ready for review January 29, 2025 09:26
@DIFSRIP DIFSRIP merged commit 3aad0dd into main Jan 30, 2025
12 checks passed
@DIFSRIP DIFSRIP deleted the fix-no-repo-error branch January 30, 2025 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ickitc ickitc ickitc approved these changes

@vlad9719 vlad9719 Awaiting requested review from vlad9719

@heraldantony heraldantony Awaiting requested review from heraldantony

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@DIFSRIP @ickitc