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(cache): can not mkdir multilevel folder #45

Closed
wants to merge 58 commits into from
Closed

fix(cache): can not mkdir multilevel folder #45

wants to merge 58 commits into from

Conversation

iamxiaojianzheng
Copy link
Contributor

about this #44

oysandvik94 and others added 30 commits June 30, 2024 20:32
Closing tab is not very determinictic, closing buffers should
hopefully always work
You can now open a global scratch buffer available in any instance,
or a named buffer which is also available in any instance
gorillamoe and others added 22 commits July 15, 2024 21:58
* Update README.md

Replace unmaintained rest.nvim with kulala.nvim.

* Update README.md

Re-naming + re-adding rest.nvim.

Similar plugins are in alphabetical order.
custom buffers can now be opened either scoped globally, or scoped to
cwd

The old method has been deprecated, and you should now use the explicit
custom global method instead. See issue #20
Removed logic around window id, simplified by always replacing the
command buffer
Collection names are now exposed through the API.
":CurlOpen collection scoped" now autocompletes on collection names
New API and usercommand methods runs a picker where you can select
collections
Result buffer now opens at the point of origin, and not to the right of
the split when the request finishes. This will probably only be relevant
when debugging.

See readme for tips on improving nvim-dap issues.
…it for execution instead

this is more inline with other plugins
@@ -21,7 +22,10 @@ local curl_cache_dir = function(custom_dir)
cache_dir = cache_dir / custom_dir ---@type Path
end

cache_dir:mkdir({ parents = true, exists_ok = true })
if vim.fn.mkdir(cache_dir:absolute(), "p") ~= 1 then
notify.error("create directory error: " .. cache_dir:absolute())
Copy link
Owner

Choose a reason for hiding this comment

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

This message goes to the user, we should perhaps formalize it a little bit:

"Error creating collection: could not create directory " .. cache_dir:absolute()

Copy link
Contributor Author

Choose a reason for hiding this comment

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

resolved

@oysandvik94
Copy link
Owner

@iamxiaojianzheng Didn't realize that it would rewrite so much, you might need to recreate this PR :P

@iamxiaojianzheng
Copy link
Contributor Author

@iamxiaojianzheng Didn't realize that it would rewrite so much, you might need to recreate this PR :P

I don't know how you modify it, I just modify the information of a commit individually through lazygit. Other commits are not affected.

@oysandvik94
Copy link
Owner

@iamxiaojianzheng I used this: https://github.com/newren/git-filter-repo

I think is rewrites the history completely, so that the email is not in the history at all.

Sorry for the complications, but maybe you need to etither sync your branch with main, or recreate your fork.

@iamxiaojianzheng
Copy link
Contributor Author

@iamxiaojianzheng I used this: https://github.com/newren/git-filter-repo

I think is rewrites the history completely, so that the email is not in the history at all.

Sorry for the complications, but maybe you need to etither sync your branch with main, or recreate your fork.

This works better for global changes, but lazygit is the easiest solution I've found so far if you're only changing partial commits. The features are really impressive.

@iamxiaojianzheng iamxiaojianzheng closed this by deleting the head repository Aug 28, 2024
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.

4 participants