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

Please also create git tags while releasing new crate versions #139

Closed
Xuanwo opened this issue Jun 23, 2022 · 10 comments
Closed

Please also create git tags while releasing new crate versions #139

Xuanwo opened this issue Jun 23, 2022 · 10 comments

Comments

@Xuanwo
Copy link
Member

Xuanwo commented Jun 23, 2022

So that users know which commits we are on.

@YangKeao
Copy link
Member

YangKeao commented Jun 23, 2022

The only maintainer of this project @YangKeao cannot push directly (so he cannot push tags). He rarely contributes to the tikv/tikv directly, and doesn't pay a lot attention to the community of tikv, so he doesn't know when/how/why it happened 😕 . He has tagged some commits a long time ago, but after some point in time, he cannot do it anymore.

Maybe when he is not that tired, he'd like to contact someone who knows the community rule of the tikv to help him solve the problem.

@Xuanwo
Copy link
Member Author

Xuanwo commented Jun 23, 2022

Maybe when he is not that tired, he'd like to contact someone who knows the community rule of the tikv to help him solve the problem.

Based on my knowledge, only tikv committers can push tags directly.

Possible solutions include as follows:

  • Inviting one of the committers to take care of this project, maybe @andylokandy? They don't need to take part in the development, just need to push tags for release. I think it's not a big burden since pprof-rs is not heavy traffic.
  • Ping one of tikv's maintainers to give you more rights on this project. Because it doesn't make sense that you are the maintainer of pprof-rs but don't have permission to push tags.
  • Transfer this project out of tikv's umbrella if the tikv community doesn't have an interest in maintaining it.

Also, ping @tisonkun for more advice about the current situation.


Databend depends heavily on pprof-rs and I'm interested in maintaining pprof-rs, we can help maintain this project.

@Xuanwo
Copy link
Member Author

Xuanwo commented Jun 23, 2022

I will raise this issue to tikv/community to get more ideas.

@Xuanwo
Copy link
Member Author

Xuanwo commented Jun 23, 2022

Ping all maintainers of tikv for comments:

@tisonkun
Copy link

@YangKeao you have Maintain access to this repository already.

@YangKeao
Copy link
Member

YangKeao commented Jun 23, 2022

@YangKeao you have Maintain access to this repository already.

Sorry for the misleading. As I just tried, I can push a tag (v0.9.1), but cannot push to master directly, is it expected?

To https://github.com/tikv/pprof-rs.git
 * [new tag]         v0.5.1 -> v0.5.1
 * [new tag]         v0.9.1 -> v0.9.1
 ! [rejected]        master -> master (fetch first)
error: failed to push some refs to 'https://github.com/tikv/pprof-rs.git'
hint: Updates were rejected because the remote contains work that you do
hint: not have locally. This is usually caused by another repository pushing
hint: to the same ref. You may want to first integrate the remote changes
hint: (e.g., 'git pull ...') before pushing again.
hint: See the 'Note about fast-forwards' in 'git push --help' for details.

I think it's fine to push the tags 😆 . I'll tag some old releases to fix this problem.

Then the release process would be: @YangKeao submits a PR to upgrade the version, after it's merged, he will tag the commit and push the tag.

@Xuanwo
Copy link
Member Author

Xuanwo commented Jun 23, 2022

As I just tried, I can push a tag (v0.9.1), but cannot push to master directly, is it expected?

I think it's expected. the master branch is protected and doesn't allow pushing directly.

Then the release process would be: @YangKeao submits a PR to upgrade the version, after it's merged, he will tag the commit and push the tag.

Looks good to me!

@skyzh
Copy link
Member

skyzh commented Jun 23, 2022

Maybe we can integrate with a GitHub Action to automatically create release (and possibly upload to crates.io?)

Some examples: https://github.com/risinglightdb/sqllogictest-rs/blob/main/.github/workflows/release.yml

@YangKeao
Copy link
Member

Maybe we can integrate with a GitHub Action to automatically create release (and possibly upload to crates.io?)

Some examples: https://github.com/risinglightdb/sqllogictest-rs/blob/main/.github/workflows/release.yml

Automatically publishing on crates.io is pretty cool, maybe I could add a bot as the owner of this crate, and publish the crate through bot's token 🤔 .

@Xuanwo
Copy link
Member Author

Xuanwo commented Jun 28, 2022

Closing for addressed, thanks @YangKeao again!

And also thanks advice from @tisonkun @skyzh.

@Xuanwo Xuanwo closed this as completed Jun 28, 2022
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

No branches or pull requests

4 participants