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

Commando updating removal version for rpcs #7395

Merged

Conversation

ShahanaFarooqui
Copy link
Collaborator

@ShahanaFarooqui ShahanaFarooqui commented Jun 14, 2024

We documented them as deprecated in v23.08 but unintentionally didn't deprecate them in the code. Thus we are starting their actual deprecation cycle from v24.08 release.

Updated removal version for commands commando-rune, commando-listrunes, and commando-blacklist.

Changelog-None.

@ShahanaFarooqui ShahanaFarooqui added this to the v24.08 milestone Jun 14, 2024
@ShahanaFarooqui ShahanaFarooqui removed the request for review from cdecker June 14, 2024 19:23
@ShahanaFarooqui ShahanaFarooqui changed the title Commando deprecate rpcs Commando updating removal date for rpcs Jun 14, 2024
@ShahanaFarooqui ShahanaFarooqui changed the title Commando updating removal date for rpcs Commando updating removal version for rpcs Jun 14, 2024
@ShahanaFarooqui ShahanaFarooqui force-pushed the commando-deprecate-rpcs branch 2 times, most recently from 6bb5a50 to 7ad45d9 Compare June 14, 2024 23:15
@rustyrussell
Copy link
Contributor

The second commit message does need to explain:

  1. We documented them as deprecated, but didn't deprecate them in the code!
  2. Thus, we need to start the actual deprecation cycle from the next release.

We documented them as deprecated in v23.08 but unintentionally didn't deprecate them in the code. Thus we are starting their actual deprecation cycle from v24.08 release.

Updating removal version for commands `commando-rune`, `commando-listrunes`, and `commando-blacklist`

Changelog-None.
@ShahanaFarooqui
Copy link
Collaborator Author

The second commit message does need to explain:

  1. We documented them as deprecated, but didn't deprecate them in the code!
  2. Thus, we need to start the actual deprecation cycle from the next release.

Thanks, updated the commit and PR message.

Copy link
Contributor

@rustyrussell rustyrussell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ack 3709af5

@rustyrussell rustyrussell merged commit 35101c0 into ElementsProject:master Jun 18, 2024
35 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.

2 participants