profile
viewpoint
Tom Milligan tommilligan @reinfer London, UK linkedin.com/in/tommilligan477 Backend, systems and CI engineer. Working to automate all the things.

reinfer/cli 3

Command line interface for Re:infer and Rust client library

tommilligan/diesel_cockroach 1

Additional Diesel ORM support for CockroachDB syntax.

reinfer/aelita 0

A mergebot for pull-request review

reinfer/rethinkdb-python 0

Python driver for RethinkDB

tommilligan/adonais 0

KEATS to Google Calendar integration.

tommilligan/alexa-knit-bot 0

Help with knitting patterns, stitches and general questions.

tommilligan/awesome-flake8-extensions 0

:octocat: A curated awesome list of flake8 extensions. Feel free to contribute! :mortar_board:

tommilligan/bash-config 0

Personalised bash config

tommilligan/bat 0

A cat(1) clone with wings.

tommilligan/behindthename-names-all 0

Quick scraper to download all name data from behindthename.com

delete branch tommilligan/adonais

delete branch : dependabot/cargo/sync/combine-4.3.2

delete time in 3 days

PR closed tommilligan/adonais

build(deps): bump combine from 4.1.0 to 4.3.2 in /sync dependencies rust

Bumps combine from 4.1.0 to 4.3.2. <details> <summary>Changelog</summary> <p><em>Sourced from <a href="https://github.com/Marwes/combine/blob/master/CHANGELOG.md">combine's changelog</a>.</em></p> <blockquote> <h3>v4.3.2 (2020-09-08)</h3> <p><!-- raw HTML omitted --><!-- raw HTML omitted --></p> <h3>v4.3.1 (2020-08-26)</h3> <h4>Bug Fixes</h4> <ul> <li>Make the decode macros work for redis (<a href="https://github.com/Marwes/combine/commit/82b908d80ac21aeab27b169a1f1857a68a6bb11c">82b908d8</a>)</li> </ul> <p><!-- raw HTML omitted --><!-- raw HTML omitted --></p> <h2>v4.3.0 (2020-07-10)</h2> <h4>Features</h4> <ul> <li>Specialize decoding on BufReader (<a href="https://github.com/Marwes/combine/commit/9559e114658b60a59d570ffd78f4d7ecb597b814">9559e114</a>)</li> <li>Add a BufReader abstraction for Decoder (<a href="https://github.com/Marwes/combine/commit/d65a23e81f482c6a5482e53d63c17fcbcf67d623">d65a23e8</a>)</li> <li>Allow tuple structs and functions in struct_parser! (<a href="https://github.com/Marwes/combine/commit/48a16f6bf84939da9eacb1f82b4cff6ca7f324e0">48a16f6b</a>)</li> </ul> <p><!-- raw HTML omitted --><!-- raw HTML omitted --></p> <h3>v4.2.1 (2020-05-20)</h3> <h4>Performance</h4> <ul> <li>Use size_hint in iterate (<a href="https://github.com/Marwes/combine/commit/b2649e3b467fc84ef48a91d56ec1fcb40291978e">b2649e3b</a>)</li> <li>Pre-allocate collections for count* parsers (<a href="https://github.com/Marwes/combine/commit/5f37857c988b81c2b0613d6f2ab14576c794ae99">5f37857c</a>)</li> </ul> <p><!-- raw HTML omitted --><!-- raw HTML omitted --></p> <h2>v4.2.0 (2020-05-17)</h2> <h4>Features</h4> <ul> <li>Add any_send_sync_partial_state (<a href="https://github.com/Marwes/combine/commit/7e90807a0949411b6aaf24a677e6530a530a1478">7e90807a</a>)</li> </ul> <p><!-- raw HTML omitted --><!-- raw HTML omitted --></p> </blockquote> </details> <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/Marwes/combine/commit/dfba2eff7a0d5021f1e316a4641e1bfcc20206f7"><code>dfba2ef</code></a> (cargo-release) version 4.3.2</li> <li><a href="https://github.com/Marwes/combine/commit/0b2965176b16fca235bd3483d1adb98c6a14397f"><code>0b29651</code></a> Updated changelog</li> <li><a href="https://github.com/Marwes/combine/commit/93ef2d5265451f036c563b6adf882a40dbff0373"><code>93ef2d5</code></a> Merge pull request <a href="https://github-redirect.dependabot.com/Marwes/combine/issues/296">#296</a> from Marwes/nits</li> <li><a href="https://github.com/Marwes/combine/commit/b2dc3f8ef90bc7719f0cff6e37286372d7072df0"><code>b2dc3f8</code></a> clippy</li> <li><a href="https://github.com/Marwes/combine/commit/bf312d9ba4352187a9d2d9df17ed5efb6e3faf53"><code>bf312d9</code></a> cargo fix --edition-idioms</li> <li><a href="https://github.com/Marwes/combine/commit/829fafcedeb7458f7c927cc46a02bd790ac1c161"><code>829fafc</code></a> Clippy suggestions</li> <li><a href="https://github.com/Marwes/combine/commit/cd681dc141ced448daf8fcb0086ff7f717e409b1"><code>cd681dc</code></a> Merge pull request <a href="https://github-redirect.dependabot.com/Marwes/combine/issues/295">#295</a> from Marwes/buf_reader</li> <li><a href="https://github.com/Marwes/combine/commit/9b8e8ecd3a5c40d109f226edd0e76d835ab2a49a"><code>9b8e8ec</code></a> (cargo-release) start next development iteration 4.3.2-alpha.0</li> <li><a href="https://github.com/Marwes/combine/commit/e5345e845949db26fe0315aba73a468708052121"><code>e5345e8</code></a> (cargo-release) version 4.3.1</li> <li><a href="https://github.com/Marwes/combine/commit/91464607c19f2e7441bf4683f8ca73869e859b4e"><code>9146460</code></a> Updated changelog</li> <li>Additional commits viewable in <a href="https://github.com/Marwes/combine/compare/v4.1.0...v4.3.2">compare view</a></li> </ul> </details> <br />

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


<details> <summary>Dependabot commands and options</summary> <br />

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  • @dependabot badge me will comment on this PR with code to add a "Dependabot enabled" badge to your readme

Additionally, you can set the following in the .dependabot/config.yml file in this repo:

  • Update frequency
  • Automerge options (never/patch/minor, and dev/runtime dependencies)
  • Out-of-range updates (receive only lockfile updates, if desired)
  • Security updates (receive only security updates, if desired)

</details>

+11 -4

1 comment

2 changed files

dependabot-preview[bot]

pr closed time in 3 days

pull request commenttommilligan/adonais

build(deps): bump combine from 4.1.0 to 4.3.2 in /sync

Superseded by #73.

dependabot-preview[bot]

comment created time in 3 days

PR opened tommilligan/adonais

build(deps): bump combine from 4.1.0 to 4.4.0 in /sync

Bumps combine from 4.1.0 to 4.4.0. <details> <summary>Changelog</summary> <p><em>Sourced from <a href="https://github.com/Marwes/combine/blob/master/CHANGELOG.md">combine's changelog</a>.</em></p> <blockquote> <h2>v4.4.0 (2020-11-18)</h2> <h4>Features</h4> <ul> <li>Add support for decoding using tokio_03 (<a href="https://github.com/Marwes/combine/commit/ce1612ffa0a15547bc967474b511ec35eaaf743f">ce1612ff</a>)</li> <li>Add a length_prefix combinator (<a href="https://github.com/Marwes/combine/commit/9e343b2a0067ea70b049369ea1023c364c14b19d">9e343b2a</a>, closes <a href="https://github-redirect.dependabot.com/Marwes/combine/issues/297">#297</a>)</li> <li>Add a spanned combinator (<a href="https://github.com/Marwes/combine/commit/1a70f3f4219eac71880cbb376e62d3f01c22d981">1a70f3f4</a>)</li> <li>Add a spanned combinator (<a href="https://github.com/Marwes/combine/commit/9a8fead85b3eb0d7ee21f80d076af9f59aa408d1">9a8fead8</a>)</li> <li>Allow conversion between errors with different token,range,position types (<a href="https://github.com/Marwes/combine/commit/65fe0af27fa560f90da2723160efd2d1782bea4f">65fe0af2</a>, breaks <a href="https://github.com/Marwes/combine/issues/">#</a>)</li> </ul> <h4>Breaking Changes</h4> <ul> <li>Allow conversion between errors with different token,range,position types (<a href="https://github.com/Marwes/combine/commit/65fe0af27fa560f90da2723160efd2d1782bea4f">65fe0af2</a>, breaks <a href="https://github.com/Marwes/combine/issues/">#</a>)</li> </ul> <p><!-- raw HTML omitted --><!-- raw HTML omitted --></p> <h3>v4.3.2 (2020-09-08)</h3> <p><!-- raw HTML omitted --><!-- raw HTML omitted --></p> <h3>v4.3.1 (2020-08-26)</h3> <h4>Bug Fixes</h4> <ul> <li>Make the decode macros work for redis (<a href="https://github.com/Marwes/combine/commit/82b908d80ac21aeab27b169a1f1857a68a6bb11c">82b908d8</a>)</li> </ul> <p><!-- raw HTML omitted --><!-- raw HTML omitted --></p> <h2>v4.3.0 (2020-07-10)</h2> <h4>Features</h4> <ul> <li>Specialize decoding on BufReader (<a href="https://github.com/Marwes/combine/commit/9559e114658b60a59d570ffd78f4d7ecb597b814">9559e114</a>)</li> <li>Add a BufReader abstraction for Decoder (<a href="https://github.com/Marwes/combine/commit/d65a23e81f482c6a5482e53d63c17fcbcf67d623">d65a23e8</a>)</li> <li>Allow tuple structs and functions in struct_parser! (<a href="https://github.com/Marwes/combine/commit/48a16f6bf84939da9eacb1f82b4cff6ca7f324e0">48a16f6b</a>)</li> </ul> <p><!-- raw HTML omitted --><!-- raw HTML omitted --></p> <h3>v4.2.1 (2020-05-20)</h3> <h4>Performance</h4> <!-- raw HTML omitted --> </blockquote> </details> <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/Marwes/combine/commit/73dac54e64f3b026567f787da33aa89c6253771c"><code>73dac54</code></a> (cargo-release) version 4.4.0</li> <li><a href="https://github.com/Marwes/combine/commit/541cef6b650ccfa50e9197585c43ac42d089e622"><code>541cef6</code></a> Updated changelog</li> <li><a href="https://github.com/Marwes/combine/commit/c52d2bb2030a27470cd9da4d0e36751477767d9a"><code>c52d2bb</code></a> Merge pull request <a href="https://github-redirect.dependabot.com/Marwes/combine/issues/306">#306</a> from Marwes/tokio_03</li> <li><a href="https://github.com/Marwes/combine/commit/e0bf1af5346723fe594ff4bd516595dbc48231fb"><code>e0bf1af</code></a> chore: Only check the build on 1.40</li> <li><a href="https://github.com/Marwes/combine/commit/d955069b5113a71e1362983e5c959ee18f8e427d"><code>d955069</code></a> Fix CombineRead features</li> <li><a href="https://github.com/Marwes/combine/commit/ce1612ffa0a15547bc967474b511ec35eaaf743f"><code>ce1612f</code></a> feat: Add support for decoding using tokio_03</li> <li><a href="https://github.com/Marwes/combine/commit/e47d45ba1178306fc4ebfa11cb091f7db916dc2b"><code>e47d45b</code></a> Merge pull request <a href="https://github-redirect.dependabot.com/Marwes/combine/issues/298">#298</a> from Marwes/length_prefix</li> <li><a href="https://github.com/Marwes/combine/commit/996383d4586a60cc968991ebf36016d3f7188496"><code>996383d</code></a> Fix no std</li> <li><a href="https://github.com/Marwes/combine/commit/c8f308f6a6a95d1f3e9d4223651e4a19cb67df59"><code>c8f308f</code></a> Merge pull request <a href="https://github-redirect.dependabot.com/Marwes/combine/issues/305">#305</a> from Marwes/span</li> <li><a href="https://github.com/Marwes/combine/commit/9e343b2a0067ea70b049369ea1023c364c14b19d"><code>9e343b2</code></a> feat: Add a length_prefix combinator</li> <li>Additional commits viewable in <a href="https://github.com/Marwes/combine/compare/v4.1.0...v4.4.0">compare view</a></li> </ul> </details> <br />

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


<details> <summary>Dependabot commands and options</summary> <br />

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  • @dependabot badge me will comment on this PR with code to add a "Dependabot enabled" badge to your readme

Additionally, you can set the following in the .dependabot/config.yml file in this repo:

  • Update frequency
  • Automerge options (never/patch/minor, and dev/runtime dependencies)
  • Out-of-range updates (receive only lockfile updates, if desired)
  • Security updates (receive only security updates, if desired)

</details>

+654 -649

0 comment

2 changed files

pr created time in 3 days

create barnchtommilligan/adonais

branch : dependabot/cargo/sync/combine-4.4.0

created branch time in 3 days

delete branch reinfer/cli

delete branch : dependabot/cargo/cli/env_logger-0.8.2

delete time in 5 days

push eventreinfer/cli

dependabot[bot]

commit sha 73d52881ea9b6ae214fdfc4f81b36a94b042c4e7

chore(deps): update env_logger requirement from 0.7.1 to 0.8.2 in /cli (#35) Updates the requirements on [env_logger](https://github.com/env-logger-rs/env_logger) to permit the latest version. - [Release notes](https://github.com/env-logger-rs/env_logger/releases) - [Changelog](https://github.com/env-logger-rs/env_logger/blob/master/CHANGELOG.md) - [Commits](https://github.com/env-logger-rs/env_logger/compare/v0.7.1...v0.8.2) Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

view details

push time in 5 days

PR merged reinfer/cli

chore(deps): update env_logger requirement from 0.7.1 to 0.8.2 in /cli dependencies

Updates the requirements on env_logger to permit the latest version. <details> <summary>Release notes</summary> <p><em>Sourced from <a href="https://github.com/env-logger-rs/env_logger/releases">env_logger's releases</a>.</em></p> <blockquote> <h2>0.8.2</h2> <p>Fixed a panic on io errors when writing to stdout / stderr (<a href="https://github-redirect.dependabot.com/env-logger-rs/env_logger/issues/184">#184</a>).</p> </blockquote> </details> <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/env-logger-rs/env_logger/commit/0f53c8dd5bf65b80d1fdb8ea0befd848f120c631"><code>0f53c8d</code></a> Release version 0.8.2</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/26e821b94f0b0b9fd3536c0ba7095c1b6c7a3ffe"><code>26e821b</code></a> Update repository links and release 0.8.1</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/9ffe00de4ff1429ccd004c42f82034b56a41a957"><code>9ffe00d</code></a> Release version 0.8.0</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/2b33c97117355cdc599aa29fb1a64b8016ee3f68"><code>2b33c97</code></a> Fix rustc & clippy lints</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/b8c3754a1de50ac720657d2f328ea98148e1d80d"><code>b8c3754</code></a> chore(examples): add syslog friendly format (<a href="https://github-redirect.dependabot.com/env-logger-rs/env_logger/issues/174">#174</a>)</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/a5a7ddc34ee82fe043eed8c3f89f5f83058f6a6f"><code>a5a7ddc</code></a> ci(docs): remove ci crate (<a href="https://github-redirect.dependabot.com/env-logger-rs/env_logger/issues/173">#173</a>)</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/3a331e46362bd96c042af9558de3a638c841d448"><code>3a331e4</code></a> chore(readme): fix badge links</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/eeedfa99be9b9619e84154de8592ce24c0f4daf5"><code>eeedfa9</code></a> ci(docs): add manual run trigger (<a href="https://github-redirect.dependabot.com/env-logger-rs/env_logger/issues/171">#171</a>)</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/2793bb5696c7740ced5c82ad27d4a9a7cd093a44"><code>2793bb5</code></a> ci(workflow): add documentation workflow (<a href="https://github-redirect.dependabot.com/env-logger-rs/env_logger/issues/170">#170</a>)</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/0ef5cffa29a45c9b0c572f470581d617fc7e50b0"><code>0ef5cff</code></a> Merge pull request <a href="https://github-redirect.dependabot.com/env-logger-rs/env_logger/issues/168">#168</a> from KodrAus/chore/build-fn-cleanup</li> <li>Additional commits viewable in <a href="https://github.com/env-logger-rs/env_logger/compare/v0.7.1...v0.8.2">compare view</a></li> </ul> </details> <br />

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


<details> <summary>Dependabot commands and options</summary> <br />

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

</details>

+5 -14

0 comment

2 changed files

dependabot[bot]

pr closed time in 5 days

Pull request review commentreinfer/cli

feature: support for uploading comments with audio

 fn upload_batch(         unchanged,     }); +    // Upload audio+    for (comment_id, audio_path) in audio_paths.iter() {

Nit: as this argument appears after annotations in the arguments list, it might be tidy if similarly this block was after the annotations block. Unless there's a logical reason not to do so?

mcobzarenco

comment created time in 5 days

Pull request review commentreinfer/cli

feature: support for uploading comments with audio

 impl Endpoints {             })     } +    fn comment_audio(&self, source_id: &SourceId, comment_id: &CommentId) -> Result<Url> {+        self.base+            .join(&format!(+                "/api/_private/sources/id:{}/comments/{}/audio",+                source_id.0, comment_id.0+            ))+            .chain_err(|| ErrorKind::Unknown {+                message: format!(+                    "Could not build sync URL for source id `{}` and comment id `{}`.",
                    "Could not build put audio URL for source id `{}` and comment id `{}`.",
mcobzarenco

comment created time in 5 days

Pull request review commentreinfer/cli

feature: support for uploading comments with audio

 pub struct NewAnnotatedComment {     pub labelling: Option<NewLabelling>,     #[serde(skip_serializing_if = "Option::is_none")]     pub entities: Option<NewEntities>,+    #[serde(skip_serializing_if = "Option::is_none")]+    pub audio_path: Option<PathBuf>,

Stupid question: I assume that this doesn't round-trip? If so, should it go in some clearly denoted subsection like "upload_extras" ?

mcobzarenco

comment created time in 5 days

push eventreinfer/cli

Tom Milligan

commit sha 573e98ae31866462e7ab89dbd94685131bfa8cbc

api/cli: add 'delete comments' to delete comments by id (#32)

view details

dependabot[bot]

commit sha f89ea33a6d1d74ed1716765daea25b26ed5a9777

chore(deps): update env_logger requirement from 0.7.1 to 0.8.2 in /cli Updates the requirements on [env_logger](https://github.com/env-logger-rs/env_logger) to permit the latest version. - [Release notes](https://github.com/env-logger-rs/env_logger/releases) - [Changelog](https://github.com/env-logger-rs/env_logger/blob/master/CHANGELOG.md) - [Commits](https://github.com/env-logger-rs/env_logger/compare/v0.7.1...v0.8.2) Signed-off-by: dependabot[bot] <support@github.com>

view details

push time in 5 days

delete branch reinfer/cli

delete branch : require-test-org

delete time in 5 days

delete branch reinfer/cli

delete branch : rm-dockerfile

delete time in 5 days

push eventreinfer/cli

David Hewitt

commit sha 1032e2e25b43b8c206cc4ee0336bd5551a0c6153

dockerfile: remove (#33)

view details

dependabot[bot]

commit sha cca18eb082818a78fe78886ade96fdb201709004

chore(deps): update env_logger requirement from 0.7.1 to 0.8.2 in /cli Updates the requirements on [env_logger](https://github.com/env-logger-rs/env_logger) to permit the latest version. - [Release notes](https://github.com/env-logger-rs/env_logger/releases) - [Changelog](https://github.com/env-logger-rs/env_logger/blob/master/CHANGELOG.md) - [Commits](https://github.com/env-logger-rs/env_logger/compare/v0.7.1...v0.8.2) Signed-off-by: dependabot[bot] <support@github.com>

view details

push time in 5 days

push eventreinfer/cli

dependabot[bot]

commit sha d1ae2cd95dd84266cda713ea927ae7fa4f251533

chore(deps): update env_logger requirement from 0.7.1 to 0.8.2 in /cli Updates the requirements on [env_logger](https://github.com/env-logger-rs/env_logger) to permit the latest version. - [Release notes](https://github.com/env-logger-rs/env_logger/releases) - [Changelog](https://github.com/env-logger-rs/env_logger/blob/master/CHANGELOG.md) - [Commits](https://github.com/env-logger-rs/env_logger/compare/v0.7.1...v0.8.2) Signed-off-by: dependabot[bot] <support@github.com>

view details

push time in 5 days

push eventreinfer/cli

David Hewitt

commit sha 1032e2e25b43b8c206cc4ee0336bd5551a0c6153

dockerfile: remove (#33)

view details

push time in 5 days

PR merged reinfer/cli

dockerfile: remove

I don't think this file is needed any more.

+0 -39

0 comment

1 changed file

davidhewitt

pr closed time in 5 days

Pull request review commentreinfer/cli

api/cli: add 'delete comments' to delete comments by id

 impl Client {      pub fn delete_dataset<IdentifierT>(&self, dataset: IdentifierT) -> Result<()>     where-        IdentifierT: Into<DatasetIdentifier> + Clone,+        IdentifierT: Into<DatasetIdentifier>,     {-        let dataset_id = match dataset.clone().into() {+        let dataset_id = match dataset.into() {             DatasetIdentifier::Id(dataset_id) => dataset_id,-            DatasetIdentifier::FullName(_) => self.get_dataset(dataset)?.id,+            dataset @ DatasetIdentifier::FullName(_) => self.get_dataset(dataset)?.id,

:+1: thanks for pushing this tidy up through the whole file!

tommilligan

comment created time in 5 days

push eventreinfer/cli

David Hewitt

commit sha 1cc6efe147e770360fb330f46524c5924a72fbe3

dockerfile: remove

view details

push time in 5 days

delete branch reinfer/cli

delete branch : dependabot/cargo/cli/env_logger-0.8.1

delete time in 5 days

PR closed reinfer/cli

chore(deps): update env_logger requirement from 0.7.1 to 0.8.1 in /cli dependencies

Updates the requirements on env_logger to permit the latest version. <details> <summary>Release notes</summary> <p><em>Sourced from <a href="https://github.com/env-logger-rs/env_logger/releases">env_logger's releases</a>.</em></p> <blockquote> <h2>0.8.1</h2> <p>Update links in the documentation that were pointing to the old repository location.</p> </blockquote> </details> <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/env-logger-rs/env_logger/commit/26e821b94f0b0b9fd3536c0ba7095c1b6c7a3ffe"><code>26e821b</code></a> Update repository links and release 0.8.1</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/9ffe00de4ff1429ccd004c42f82034b56a41a957"><code>9ffe00d</code></a> Release version 0.8.0</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/2b33c97117355cdc599aa29fb1a64b8016ee3f68"><code>2b33c97</code></a> Fix rustc & clippy lints</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/b8c3754a1de50ac720657d2f328ea98148e1d80d"><code>b8c3754</code></a> chore(examples): add syslog friendly format (<a href="https://github-redirect.dependabot.com/env-logger-rs/env_logger/issues/174">#174</a>)</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/a5a7ddc34ee82fe043eed8c3f89f5f83058f6a6f"><code>a5a7ddc</code></a> ci(docs): remove ci crate (<a href="https://github-redirect.dependabot.com/env-logger-rs/env_logger/issues/173">#173</a>)</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/3a331e46362bd96c042af9558de3a638c841d448"><code>3a331e4</code></a> chore(readme): fix badge links</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/eeedfa99be9b9619e84154de8592ce24c0f4daf5"><code>eeedfa9</code></a> ci(docs): add manual run trigger (<a href="https://github-redirect.dependabot.com/env-logger-rs/env_logger/issues/171">#171</a>)</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/2793bb5696c7740ced5c82ad27d4a9a7cd093a44"><code>2793bb5</code></a> ci(workflow): add documentation workflow (<a href="https://github-redirect.dependabot.com/env-logger-rs/env_logger/issues/170">#170</a>)</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/0ef5cffa29a45c9b0c572f470581d617fc7e50b0"><code>0ef5cff</code></a> Merge pull request <a href="https://github-redirect.dependabot.com/env-logger-rs/env_logger/issues/168">#168</a> from KodrAus/chore/build-fn-cleanup</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/b24920eb7afaaa95edca0dd36f50caaca2b95ccf"><code>b24920e</code></a> deprecate env_logger::from_env and clean up examples</li> <li>Additional commits viewable in <a href="https://github.com/env-logger-rs/env_logger/compare/v0.7.1...v0.8.1">compare view</a></li> </ul> </details> <br />

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


<details> <summary>Dependabot commands and options</summary> <br />

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

</details>

+1 -1

1 comment

1 changed file

dependabot[bot]

pr closed time in 5 days

pull request commentreinfer/cli

chore(deps): update env_logger requirement from 0.7.1 to 0.8.1 in /cli

Superseded by #35.

dependabot[bot]

comment created time in 5 days

PR opened reinfer/cli

chore(deps): update env_logger requirement from 0.7.1 to 0.8.2 in /cli

Updates the requirements on env_logger to permit the latest version. <details> <summary>Release notes</summary> <p><em>Sourced from <a href="https://github.com/env-logger-rs/env_logger/releases">env_logger's releases</a>.</em></p> <blockquote> <h2>0.8.2</h2> <p>Fixed a panic on io errors when writing to stdout / stderr (<a href="https://github-redirect.dependabot.com/env-logger-rs/env_logger/issues/184">#184</a>).</p> </blockquote> </details> <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/env-logger-rs/env_logger/commit/0f53c8dd5bf65b80d1fdb8ea0befd848f120c631"><code>0f53c8d</code></a> Release version 0.8.2</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/26e821b94f0b0b9fd3536c0ba7095c1b6c7a3ffe"><code>26e821b</code></a> Update repository links and release 0.8.1</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/9ffe00de4ff1429ccd004c42f82034b56a41a957"><code>9ffe00d</code></a> Release version 0.8.0</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/2b33c97117355cdc599aa29fb1a64b8016ee3f68"><code>2b33c97</code></a> Fix rustc & clippy lints</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/b8c3754a1de50ac720657d2f328ea98148e1d80d"><code>b8c3754</code></a> chore(examples): add syslog friendly format (<a href="https://github-redirect.dependabot.com/env-logger-rs/env_logger/issues/174">#174</a>)</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/a5a7ddc34ee82fe043eed8c3f89f5f83058f6a6f"><code>a5a7ddc</code></a> ci(docs): remove ci crate (<a href="https://github-redirect.dependabot.com/env-logger-rs/env_logger/issues/173">#173</a>)</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/3a331e46362bd96c042af9558de3a638c841d448"><code>3a331e4</code></a> chore(readme): fix badge links</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/eeedfa99be9b9619e84154de8592ce24c0f4daf5"><code>eeedfa9</code></a> ci(docs): add manual run trigger (<a href="https://github-redirect.dependabot.com/env-logger-rs/env_logger/issues/171">#171</a>)</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/2793bb5696c7740ced5c82ad27d4a9a7cd093a44"><code>2793bb5</code></a> ci(workflow): add documentation workflow (<a href="https://github-redirect.dependabot.com/env-logger-rs/env_logger/issues/170">#170</a>)</li> <li><a href="https://github.com/env-logger-rs/env_logger/commit/0ef5cffa29a45c9b0c572f470581d617fc7e50b0"><code>0ef5cff</code></a> Merge pull request <a href="https://github-redirect.dependabot.com/env-logger-rs/env_logger/issues/168">#168</a> from KodrAus/chore/build-fn-cleanup</li> <li>Additional commits viewable in <a href="https://github.com/env-logger-rs/env_logger/compare/v0.7.1...v0.8.2">compare view</a></li> </ul> </details> <br />

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


<details> <summary>Dependabot commands and options</summary> <br />

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

</details>

+1 -1

0 comment

1 changed file

pr created time in 5 days

create barnchreinfer/cli

branch : dependabot/cargo/cli/env_logger-0.8.2

created branch time in 5 days

push eventreinfer/cli

David Hewitt

commit sha 267dbc78b0e56047f712a37d784c2a190043917e

tests: require integration tests to be explicit about organisation (#34)

view details

David Hewitt

commit sha 33a17052d32438b9695b2ccd326974b8e435b88c

Merge branch 'master' into rm-dockerfile

view details

push time in 5 days

push eventreinfer/cli

David Hewitt

commit sha 267dbc78b0e56047f712a37d784c2a190043917e

tests: require integration tests to be explicit about organisation (#34)

view details

push time in 5 days

PR merged reinfer/cli

tests: require integration tests to be explicit about organisation

At the moment to be "smart" the integration tests try to default the organisation to be the same as the username.

Almost always a user doesn't have an organisation which matches their username, so this often fails.

Also implicit behaviour is probably not good. These tests now require the env variable to be set explicitly.

+5 -18

0 comment

2 changed files

davidhewitt

pr closed time in 5 days

PR opened reinfer/cli

tests: require integration tests to be explicit about organisation

At the moment to be "smart" the integration tests try to default the organisation to be the same as the username.

Almost always a user doesn't have an organisation which matches their username, so this often fails.

Also implicit behaviour is probably not good. These tests now require the env variable to be set explicitly.

+5 -18

0 comment

2 changed files

pr created time in 5 days

push eventreinfer/cli

David Hewitt

commit sha 8c7101a17b8c83f93bb9cd0bc1f067788d18196f

tests: require integration tests to be explicit about organisation

view details

push time in 5 days

more