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.

Sign up for GitHub

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

chore: update sprocket to latest WDL crate #46

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from
Draft

chore: update sprocket to latest WDL crate #46

Show file tree
Hide file tree
Changes from 9 commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
b3dbcee
chore: update sprocket to latest WDL crate
a-frantz Dec 17, 2024
d0d5b0a
feat: --local-only and --single-document args to check
a-frantz Dec 18, 2024
4ebabcd
Update CHANGELOG.md
a-frantz Dec 18, 2024
a35bf8b
Update CHANGELOG.md
a-frantz Dec 18, 2024
6e1c196
chore: cargo update
a-frantz Dec 19, 2024
c1cbd2b
chore: fmt
a-frantz Dec 19, 2024
cab0c02
feat: add shellcheck
a-frantz Dec 20, 2024
b061ef1
docs: label shellcheck experimental
a-frantz Dec 20, 2024
2b81661
Update Cargo.lock
a-frantz Dec 30, 2024
d2d7353
chore: update to latest
a-frantz Jan 9, 2025
faf1dbf
chore: clippy and dmt
a-frantz Jan 9, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 11 additions & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## Unreleased

### Changed

* Updated WDL crate to latest. This adds support for
Copy link
Member

Choose a reason for hiding this comment

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

Didn't Clay want to keep sprocket pinned to a release?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, but that release doesn't exist yet. This PR will be updated to pull an official release once one exists.

I think the language here is fine? Maybe it should specify the version of "latest" once that is out?

Copy link
Member

Choose a reason for hiding this comment

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

OK, I would swap latest with the version when it is known.

checking/linting remote URLs ([#46](https://github.com/stjude-rust-labs/sprocket/pull/46)).
* Only allow one `file` argument to `check/lint` instead of any number of local files and directories
([#46](https://github.com/stjude-rust-labs/sprocket/pull/46)).

### Added

* Added `--local-only` and `--single-document` args to `check/lint` ([#46](https://github.com/stjude-rust-labs/sprocket/pull/46)).

## 0.9.0 - 10-22-2024

### Changed
Expand Down
Loading
Loading