Skip to content

Added Gura support#446

Closed
mkvolkov wants to merge 5 commits into
rust-cli:masterfrom
mkvolkov:gura
Closed

Added Gura support#446
mkvolkov wants to merge 5 commits into
rust-cli:masterfrom
mkvolkov:gura

Conversation

@mkvolkov

Copy link
Copy Markdown

Added support for Gura. Closes issue #245

@matthiasbeyer matthiasbeyer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hi! Thanks for continuing!

I am off the grid for approx. The next week, so please expect delays in response.

One question (see review). There might be more, I cannot do in depth reviews now as I am on a train. Either way, thanks again for contributing!

Comment thread Cargo.toml Outdated
Comment thread src/file/format/mod.rs Outdated
Comment thread src/file/format/mod.rs Outdated
Comment thread src/file/format/mod.rs
Comment thread src/file/format/mod.rs Outdated
@polarathene

Copy link
Copy Markdown
Contributor

@matthiasbeyer friendly ping as this PR is getting stale (already conflicts raised).

Only one ura remains from your feedback, but it's intentional as it's the file format extension.

@matthiasbeyer

Copy link
Copy Markdown
Member

Yes this needs a rebase for resolving conflicts.

@polarathene

polarathene commented Oct 4, 2023

Copy link
Copy Markdown
Contributor

Yes this needs a rebase for resolving conflicts.

Done #467

@mkvolkov can update their branch here, otherwise my alternative one should be sufficient if they're unresponsive.

@mkvolkov mkvolkov closed this by deleting the head repository Feb 11, 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.

3 participants