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

Remove redundant row #493

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mattoshibtc
Copy link

Pull Request Checklist

  • The documents updated are not in the docs/ directory. These files are
    synced from upstream repositories (lnd, lit, loop, pool and faraday), and
    should be updated in their parent repo.

@guggero
Copy link
Member

guggero commented Apr 11, 2023

Why do you think that line is redundant?

@mattoshibtc
Copy link
Author

It's the same information as the UTXO row and not consistent with the description of Timelocks on the same page: "There are absolute and relative timelocks, existing on the transactional and UTXO level".

@Liongrass
Copy link
Contributor

It's not exactly redundant. The original intention was to move it into a single line, it just looked terribly

@mattoshibtc
Copy link
Author

Yea, not completely redundant, I just think it would be a little more clear if it was removed. It definitely tripped me up when I was learning. Thanks for looking at this, no worries if ya'll want to close the PR.

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