Skip to content

prep stapi-fastapi release v0.7.0 #96

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

Jump to bottom
Merged
jkeifer merged 3 commits into from
Apr 19, 2025
Merged

prep stapi-fastapi release v0.7.0 #96

jkeifer merged 3 commits into from
Apr 19, 2025

Conversation

@jkeifer
Copy link
Member

@jkeifer jkeifer commented Apr 17, 2025

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
Copy link
Member

@gadomski gadomski left a comment

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
Copy link
Member Author

jkeifer commented Apr 18, 2025

@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
Copy link
Member

gadomski commented Apr 18, 2025

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
Copy link
Member

@gadomski gadomski left a comment

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
4 checks passed
@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.

3 participants

@jkeifer @gadomski