Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(agents): Rename CommandError::FailedWithOutput to CommandError::NonZeroExit #484

Merged
merged 1 commit into from
Dec 16, 2024

Conversation

timonv
Copy link
Member

@timonv timonv commented Dec 12, 2024

Better describes what is going on. I.e. rg exits with 1 if nothing is
found, tests generally do the same if they fail.

…r::NonZeroExit`

Better describes what is going on. I.e. `rg` exits with 1 if nothing is
found, tests generally do the same if they fail.
@timonv timonv merged commit d255772 into master Dec 16, 2024
7 checks passed
@timonv timonv deleted the feat/rename-command-error branch December 16, 2024 13:34
This was referenced Dec 17, 2024
timonv pushed a commit that referenced this pull request Dec 23, 2024
## 🤖 New release
* `swiftide`: 0.14.4 -> 0.15.0 (⚠️ API breaking changes)
* `swiftide-agents`: 0.14.4 -> 0.15.0 (✓ API compatible changes)
* `swiftide-core`: 0.14.4 -> 0.15.0 (⚠️ API breaking changes)
* `swiftide-macros`: 0.14.4 -> 0.15.0
* `swiftide-integrations`: 0.14.4 -> 0.15.0 (✓ API compatible changes)
* `swiftide-indexing`: 0.14.4 -> 0.15.0
* `swiftide-query`: 0.14.4 -> 0.15.0 (✓ API compatible changes)

### ⚠️ `swiftide` breaking changes

```
--- failure module_missing: pub module removed or renamed ---

Description:
A publicly-visible module cannot be imported by its prior path. A `pub use` may have been removed, or the module may have been renamed, removed, or made non-public.
        ref: https://doc.rust-lang.org/cargo/reference/semver.html#item-remove
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.38.0/src/lints/module_missing.ron

Failed in:
  mod swiftide::agents, previously in file /tmp/.tmpqenghN/swiftide/src/lib.rs:108
```

### ⚠️ `swiftide-core` breaking changes

```
--- failure enum_variant_added: enum variant added on exhaustive enum ---

Description:
A publicly-visible enum without #[non_exhaustive] has a new variant.
        ref: https://doc.rust-lang.org/cargo/reference/semver.html#enum-variant-new
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.38.0/src/lints/enum_variant_added.ron

Failed in:
  variant CommandError:NonZeroExit in /tmp/.tmpm3E1mt/swiftide/swiftide-core/src/agent_traits.rs:29
  variant CommandError:NonZeroExit in /tmp/.tmpm3E1mt/swiftide/swiftide-core/src/agent_traits.rs:29

--- failure enum_variant_missing: pub enum variant removed or renamed ---

Description:
A publicly-visible enum has at least one variant that is no longer available under its prior name. It may have been renamed or removed entirely.
        ref: https://doc.rust-lang.org/cargo/reference/semver.html#item-remove
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.38.0/src/lints/enum_variant_missing.ron

Failed in:
  variant CommandError::FailedWithOutput, previously in file /tmp/.tmpqenghN/swiftide-core/src/agent_traits.rs:29
  variant CommandError::FailedWithOutput, previously in file /tmp/.tmpqenghN/swiftide-core/src/agent_traits.rs:29
```

<details><summary><i><b>Changelog</b></i></summary><p>

## `swiftide`
<blockquote>

##
[0.15.0](v0.14.4...v0.15.0)
- 2024-12-23

### New features

-
[a1b9a2d](a1b9a2d)
*(query)* Ensure concrete names for transformations are used when
debugging (#496)

-
[7779c44](7779c44)
*(query)* Ensure query pipeline consistently debug logs in all other
stages too

-
[55dde88](55dde88)
*(query)* Debug full retrieved documents when debug mode is enabled
(#495)

-
[66031ba](66031ba)
*(query)* Log query pipeline answer on debug (#497)

### Miscellaneous

-
[d255772](d255772)
*(agents)* Rename `CommandError::FailedWithOutput` to
`CommandError::NonZeroExit` (#484)

````text
Better describes what is going on. I.e. `rg` exits with 1 if nothing is
  found, tests generally do the same if they fail.
````

- [0000000](0000000) Update
Cargo.toml dependencies


**Full Changelog**:
0.14.4...0.15.0
</blockquote>


</p></details>

---
This PR was generated with
[release-plz](https://github.com/release-plz/release-plz/).
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.

1 participant