Skip to content

prep stapi-fastapi release v0.7.0 - #96

Merged
jkeifer merged 3 commits into
mainfrom
jak/release/stapi-fastapi-v0.7.0
Apr 19, 2025
Merged

prep stapi-fastapi release v0.7.0#96
jkeifer merged 3 commits into
mainfrom
jak/release/stapi-fastapi-v0.7.0

Conversation

@jkeifer

@jkeifer jkeifer commented Apr 17, 2025

Copy link
Copy Markdown
Member

What I'm changing

  • Update stapi-fastapi CHANGELOG to prep v0.7.0 release

How I did it

  • n/a

Checklist

  • Tests pass: uv run pytest
  • Checks pass: uv run pre-commit run --all-files
  • CHANGELOG is updated (if necessary)

@jkeifer
jkeifer requested a review from gadomski as a code owner April 17, 2025 17:23
gadomski
gadomski requested changes Apr 18, 2025
View reviewed changes

@gadomski gadomski left a comment

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.

Need to update pyproject.toml as well:

pystapi/stapi-fastapi/pyproject.toml

Lines 3 to 4 in 1cbfba2

version = "0.6.0"
description = "Sensor Tasking API (STAPI) with FastAPI"

You let me alone in the sprint room so I did the thing you don't like with the versions 😉

@jkeifer
jkeifer force-pushed the jak/release/stapi-fastapi-v0.7.0 branch from 1cbfba2 to aa86d66 Compare April 18, 2025 17:37
@jkeifer

jkeifer commented Apr 18, 2025

Copy link
Copy Markdown
Member Author

@gadomski I thought that the release script updated pyproject.toml, and that would be run after this PR is merged? I'm trying to follow RELEASING.md, if that is not accurate let me know and I can update it too.

@gadomski

gadomski commented Apr 18, 2025

Copy link
Copy Markdown
Member

I'm trying to follow RELEASING.md, if that is not accurate let me know and I can update it too.

Nope, I missed that step when I wrote it -- updated. Sorry 'bout that.

@jkeifer
jkeifer requested a review from gadomski April 18, 2025 21:37
gadomski
gadomski approved these changes Apr 18, 2025
View reviewed changes

@gadomski gadomski left a comment

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.

LGTM! I'll let you release to make sure I'm not the only one who has :-P

@jkeifer
jkeifer merged commit 5c770e3 into main Apr 19, 2025
@jkeifer
jkeifer deleted the jak/release/stapi-fastapi-v0.7.0 branch April 19, 2025 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gadomski gadomski gadomski approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jkeifer @gadomski