Skip to content

Import YAML session exports#389

Open
GreyforgeLabs wants to merge 1 commit into
CortexLM:mainfrom
GreyforgeLabs:import-yaml-session-exports
Open

Import YAML session exports#389
GreyforgeLabs wants to merge 1 commit into
CortexLM:mainfrom
GreyforgeLabs:import-yaml-session-exports

Conversation

@GreyforgeLabs
Copy link
Copy Markdown

Summary

  • Allow cortex import to parse session exports as JSON or YAML.
  • Preserve helpful parse diagnostics for invalid content while updating them to mention JSON/YAML exports.
  • Add a binary integration regression that imports a YAML session export produced in the same shape as cortex export --format yaml.

Fixes PlatformNetwork/bounty-challenge#44938.

Validation

  • cargo test -p cortex-cli --test import_yaml --no-default-features --features cortex-tui -- --nocapture
  • cargo check -p cortex-cli --no-default-features --features cortex-tui
  • cargo fmt --package cortex-cli --check
  • git diff --check

Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

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.

[BUG] [v0.0.7] cortex export --format yaml produces files that cortex import rejects

1 participant