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

LSPS0: underscores in method names #49

Merged

Conversation

JssDWt
Copy link
Contributor

@JssDWt JssDWt commented Aug 11, 2023

Use underscores in method names in LSPS0, for consistency throughout the spec.

Main edit: Changes lsps0.listprotocols to lsps0.list_protocols

@SeverinAlexB
Copy link
Collaborator

100%

LSPS0/README.md Show resolved Hide resolved
Use underscores in method names in LSPS0, for consistency throughout the spec.
@JssDWt JssDWt force-pushed the underscored_methods branch from cb872ec to ebc8ab5 Compare August 14, 2023 10:43
@JssDWt
Copy link
Contributor Author

JssDWt commented Aug 14, 2023

Added:

method names MUST be in snake_case, i.e. words are lower
case and separated by _ characters.

@ZmnSCPxj-jr
Copy link
Contributor

This seems an uncontroversial change, OK with merging even outside of LSPS meeting.

@rbndg rbndg merged commit d5d7039 into BitcoinAndLightningLayerSpecs:main Aug 17, 2023
ErikDeSmedt added a commit to ErikDeSmedt/greenlight that referenced this pull request Aug 17, 2023
cdecker pushed a commit to ErikDeSmedt/greenlight that referenced this pull request Aug 21, 2023
cdecker pushed a commit to Blockstream/greenlight that referenced this pull request Aug 21, 2023
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.

5 participants