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 bogus error message #1283

Merged
merged 1 commit into from May 16, 2024
Merged

Fix bogus error message #1283

merged 1 commit into from May 16, 2024

Conversation

alerque
Copy link
Contributor

@alerque alerque commented Oct 4, 2023

This PR is related to #1199, but does not fix it. It only corrects the error message: the operation is an attempt to decrypt and is failing, but the error thrown says something about encrypting. Whatever is causing the error remanis unresolved, but at least the confusing/contradictory message can be fixed.

@vercel
Copy link

vercel bot commented Oct 4, 2023

Deployment failed with the following error:

The provided GitHub repository does not contain the requested branch or commit reference. Please ensure the repository is not empty.

@alerque
Copy link
Contributor Author

alerque commented Oct 4, 2023

@vercel You feeling alright buddy or have you had a glass or two too many?

@tessus
Copy link
Contributor

tessus commented May 14, 2024

This PR needs an update. The files are now under the crates dir and the line has changed to 143

@tessus
Copy link
Contributor

tessus commented May 15, 2024

LGTM.

@ellie sorry to bug you. this PR might have been forgotten, but is a low hanging fruit to reduce the number of open PRs.

Copy link
Member

@ellie ellie left a comment

Choose a reason for hiding this comment

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

Thanks @tessus! This is for sync v1 though, so will be gone fairly soon. But good to merge anyway

@ellie ellie merged commit 2e35af1 into atuinsh:main May 16, 2024
14 checks passed
@alerque alerque deleted the decrypt branch May 16, 2024 08:58
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.

None yet

3 participants