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

Prepare for initial re-release #30

Merged
merged 1 commit into from
Jan 22, 2025
Merged

Prepare for initial re-release #30

merged 1 commit into from
Jan 22, 2025

Conversation

einarmo
Copy link
Contributor

@einarmo einarmo commented Jan 22, 2025

Create a couple of scripts using cargo-release to simplify bumping versions and release management. Since we have so many crates it's a little hairy, so I added some scripts to make it more manageable.

I also wrote a rough changelog on the keep-a-changelog format. The old changelog is a little confusing in context, and uses a different format, so I moved it into docs as old_changelog.md.

Set versions on all the crates we intend to publish for now. In the future we may want to find a good way to publish opcua-codegen too, but since it's a binary, not a crate, we will have to look into publishing it as a cargo tool or something along those lines.

The first release will need to be done manually. Not only are we exceeding the per minute limit of new crates (5), but cargo-release doesn't play that well with new releases of interdependent crates. I hope to create a proper release pipeline to automate this in the future.

@oroulet
Copy link
Member

oroulet commented Jan 22, 2025

@einarmo not the right place to discuss it , but you shold rename the repository to async-opcua too so it is possible to google it knowning the crate name

@oroulet
Copy link
Member

oroulet commented Jan 22, 2025

I also hope you have a script so you do not need to do all of that for every release...

oroulet
oroulet previously approved these changes Jan 22, 2025
@einarmo
Copy link
Contributor Author

einarmo commented Jan 22, 2025

The changelog needs a script, but I committed one for bumping all the version numbers, using cargo-release.

We should probably rename the repo, yes. async-opcua-rust maybe? Calling it just async-opcua is maybe a little confusing?

@svanharmelen
Copy link
Contributor

Calling it just async-opcua is maybe a little confusing?

Maybe... But then again the Python repo is also just called opcua-asyncio and the C one is called freeopcua so 🤷🏻‍♂️

@einarmo
Copy link
Contributor Author

einarmo commented Jan 22, 2025

Maybe... But then again the Python repo is also just called opcua-asyncio and the C one is called freeopcua so 🤷🏻‍♂️

Fair enough. Let's rename the repo. I'll do that before actually cutting the release so that the metadata on crates.io isn't instantly out of sync.

@einarmo
Copy link
Contributor Author

einarmo commented Jan 22, 2025

Looks like the default rust version on github actions went up to 1.84 after I made the PR, fixed it in #32.

Create a couple of scripts using cargo-release to simplify bumping
versions and release management. Since we have so many crates it's a
little hairy, so I added some scripts to make it more manageable.

Set versions on all the crates we intend to publish for now. In the
future we may want to find a good way to publish opcua-codegen too, but
since it's a binary, not a crate, we will have to look into publishing
it as a cargo tool or something along those lines.

The first release will need to be done manually. Not only are we
exceeding the per minute limit of new crates (5), but cargo-release
doesn't play that well with new releases of interdependent crates. I
hope to create a proper release pipeline to automate this in the future.
@einarmo einarmo merged commit 7c4ce2f into master Jan 22, 2025
6 checks passed
@einarmo einarmo deleted the release-pipeline branch January 22, 2025 16:52
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.

3 participants