Skip to content

Add additional metadata#76

Merged
richfitz merged 6 commits into
mainfrom
mrc-6513
Jun 16, 2025
Merged

Add additional metadata#76
richfitz merged 6 commits into
mainfrom
mrc-6513

Conversation

@richfitz

@richfitz richfitz commented Jun 9, 2025

Copy link
Copy Markdown
Member

We need a location entry for orderly2 to be able to interact with the outpack archive, at least for things like migrating data. This PR adds the "local" location, which orderly treats specially (to the point where it cannot add it).

Metadata on initialisation, after this PR:

{
  "core": {
    "hash_algorithm": "sha256",
    "path_archive": null,
    "use_file_store": true,
    "require_complete_tree": false
  },
  "location": [
    {
      "name": "local",
      "type": "local",
      "args": {}
    }
  ]
}

Currently the location section is the empty list.

Comment thread Cargo.toml Outdated
[package]
name = "outpack"
version = "0.3.2"
version = "0.3.3"

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.

This is unrelated to this PR, but in the past I've done explicit PRs for bumping the version number, following https://github.com/mrc-ide/outpack_server?tab=readme-ov-file#releasing

I would be fine with using the same workflow we do in our R stuff, and bump the version on every commit. We'd want to tweak our CI to a) ensure the version has been bumped and b) push to PyPI on every push to main, not just on tags.

Comment thread src/config.rs Outdated
Comment thread src/config.rs Outdated
@richfitz richfitz merged commit 1bf5911 into main Jun 16, 2025
17 checks passed
plietar added a commit to mrc-ide/pyorderly that referenced this pull request Oct 30, 2025
null was invalid according to our schema, and since
mrc-ide/outpack_server#76 outpack_server has
become stricter about this since it has started deserializing that
field.

This was causing tests to fail when run against the more recent build of
outpack_server.
plietar added a commit to mrc-ide/pyorderly that referenced this pull request Oct 30, 2025
null was invalid according to our schema, and since
mrc-ide/outpack_server#76 outpack_server has
become stricter about this since it has started deserializing that
field.

This was causing tests to fail when run against the more recent build of
outpack_server.
plietar added a commit to mrc-ide/pyorderly that referenced this pull request Oct 30, 2025
null was invalid according to our schema, and since
mrc-ide/outpack_server#76 outpack_server has
become stricter about this since it has started deserializing that
field.

This was causing tests to fail when run against the more recent build of
outpack_server.
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.

2 participants