Skip to content

Conversation

@jdetter
Copy link
Collaborator

@jdetter jdetter commented Feb 28, 2025

Description of Changes

Please describe your change, mention any related tickets, and so on here.

These are the changes that I made to the publish script while doing this release. I've made the following improvements:

  • I've changed the script so that regardless if a crate is new we will try to add owners from the list.
  • I've added myself as an owner
  • Changed emails to usernames because for some reason the emails weren't working for me and also usernames are more discrete.

Besides the publish script changes I also added LICENSE files to crates that needed them in order to publish.

API and ABI breaking changes

NA

Expected complexity level and risk

1 - this is just a publish script change + adding license files

How complicated do you think these changes are? Grade on a scale from 1 to 5,
where 1 is a trivial change, and 5 is a deep-reaching and complex change.

This complexity rating applies not only to the complexity apparent in the diff,
but also to its interactions with existing and future code.

If you answered more than a 2, explain what is complex about the PR,
and what other components it interacts with in potentially concerning ways.

Testing

  • I ran the publish script and it made it all the way through

@jdetter jdetter force-pushed the jdetter/post-release-fixes branch from 6a39145 to 4142792 Compare March 3, 2025 16:30
@jdetter jdetter added this pull request to the merge queue Mar 4, 2025
Merged via the queue into master with commit 08cbd94 Mar 4, 2025
14 checks passed
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