Skip to content

feat(config): dump to path#107

Open
Nelfimov wants to merge 2 commits intoglide-wm:mainfrom
Nelfimov:feat/config-dump
Open

feat(config): dump to path#107
Nelfimov wants to merge 2 commits intoglide-wm:mainfrom
Nelfimov:feat/config-dump

Conversation

@Nelfimov
Copy link
Contributor

closes #106

@tmandry
Copy link
Contributor

tmandry commented Jan 26, 2026

See #106 (comment).

@tmandry
Copy link
Contributor

tmandry commented Mar 5, 2026

@Nelfimov are you still working on the suggestion in #106 (comment)? Maybe it would make sense to close this PR and open a new one for that.

Copy link
Contributor

@tmandry tmandry left a comment

Choose a reason for hiding this comment

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

A couple comments on the current branch in case you do pursue #106 (comment) based on it

}

fn config_is_dumped() {
let file_name = "./glide.test.toml";
Copy link
Contributor

Choose a reason for hiding this comment

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

Glide depends on the tempfile crate for making temp test files. Please use that here.


# macOS
.DS_Store/
.DS_Store
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this on purpose?

@Nelfimov
Copy link
Contributor Author

Nelfimov commented Mar 5, 2026

@Nelfimov are you still working on the suggestion in #106 (comment)?

I wanted to finish it but lately was preparing for vacation so had to postpone. I plan to come back to later this month. It totally makes sense to start from scratch.

Nevertheless wanted to say that I daily drive glide and am totally happy with it.

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.

[Feature] Dumping current config

2 participants