Skip to content

Bump version to v0.14.14.1 Spark Names #1550

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

Merged
merged 1 commit into from
Apr 19, 2025
Merged

Bump version to v0.14.14.1 Spark Names #1550

merged 1 commit into from
Apr 19, 2025

Conversation

justanwar
Copy link
Member

No description provided.

Copy link

coderabbitai bot commented Apr 9, 2025

Walkthrough

The pull request updates the build version information for the client. In both configure.ac and src/clientversion.h, the build version constant has been incremented from 0 to 1, indicating that a new build version is being released without any other changes to the software logic.

Changes

Files Change Summary
configure.ac, src/clientversion.h Incremented the build version constant from 0 to 1 in both files.

Possibly related PRs

Suggested reviewers

  • psolstice
  • ajaydono

Poem

Hop, hop, in the code I play,
Incrementing numbers in a magical way.
From 0 to 1, the build takes flight,
A rabbit’s joy in versioning delight.
🐰✨ Code bounces on—bright and light!


📜 Recent review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between cfb38bc and 399a308.

📒 Files selected for processing (2)
  • configure.ac (1 hunks)
  • src/clientversion.h (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (1)
  • GitHub Check: Create Source Distribution
🔇 Additional comments (2)
src/clientversion.h (1)

20-20: Version bump appears correct

The increment of CLIENT_VERSION_BUILD from 0 to 1 properly reflects the new build version referenced in the PR title (v0.14.14.1).

configure.ac (1)

6-6: Version bump is consistent with clientversion.h

The increment of _CLIENT_VERSION_BUILD from 0 to 1 correctly matches the change made in src/clientversion.h, maintaining consistency in version numbering across the codebase.

✨ Finishing Touches
  • 📝 Generate Docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Generate unit testing code for this file.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai generate unit testing code for this file.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and generate unit testing code.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai generate docstrings to generate docstrings for this PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai plan to trigger planning for file edits and PR creation.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

Copy link
Contributor

@ajaydono ajaydono left a comment

Choose a reason for hiding this comment

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

LGTM

@reubenyap reubenyap merged commit 7f56324 into master Apr 19, 2025
11 checks passed
@reubenyap reubenyap deleted the v014141 branch April 19, 2025 05:39
aleflm added a commit that referenced this pull request Apr 23, 2025
Rebase includes:

* UI improvements (#1475)

* Lelantus UI stripped

* More improvements

* Fixed UI hang issue during batch verification

* QT hanging improvements

* QT hanging fix

* Review commits applied

* coderabbit comments applied

* more coderabbitai comments resolved

---------

Co-authored-by: firstcryptoman <firstcryptoman@gmail.com>

* Spark names (#1532)

* Initial spark name architecture

* Spark address ownership proofs implemented.

* Missing files added

* Check the ownership proof for spark names: initial implementation

* Fixes to the core part of spark names

* Added additional field (core)

* Consensus parameters for spark names

* Fixed mempool bug

* Fixes in spark name conflict resolution

* RPCs for spark names

* Additional API for spark names tx creation

* Changed way of checking spark name tx

* Wallet API for spark name transaction creation

* API changes for spark name tx creation

* Added registersparkname RPC call

* Spark activation check for RPC

* Make spark names case-insensitive

* Spark name RPC fixes

* Faster lookup for spark name by spark address

* Fixes for spark name/address lookup

* Improvements for duplicated address detection

* Fixes for spark name state

* Block index entries for spark names

* Make dot (.) a legit symbol in spark name

* Spark name block number for testnet

* Fixed restoration of old spark name data if block is disconnected

* API for quick check of spark name transaction validity before the creation

* added isAddressMine function

* Check if the address belongs to the wallet before creating spark name transaction

* Fixed fee calculation for spark name

* Fix for spark names RPC

* Added ability to spend to spark names in "sparkspend" RPC

* UI fixes

* Additional validations

* Fix for crash with spend to unregistered name

* Fixed fee value when registering spark name for more than one year

* Spark name UI improvements

* UI modifications for sending to spark name

* Address book fixes related to spark names

* Fixed period of validity when creating spark name through GUI

* Extended spark name error info for RPC

* Fixed crash on non-HD wallet

* Request wallet password for spark name creation

* Fixed fee calculation for the spark name tx

* Fixed detection of previously used spark address for a spark name

* Unit test for spark names

* Additional unit tests

* Fixes #1533

* getsparknamedata RPC now returns JSON object

* Added "My own spark names" to the dropdown list in address book

* Added an option of displaying only own spark names for RPC. Closes #1535

* Set HF block for spark names

* Fixed a check for spark name block start

* Added tests for correctly respecting HF block number

* Check if we're over HF before spark name transaction creation

* new rpc for spark name (#1552)

* Fixed spark name tests

* Changed HF date

* Change of HF block number

---------

Co-authored-by: levonpetrosyan93 <petrosyan.levon93@gmail.com>
Co-authored-by: levoncrypto <levoncrypto1994@gmail.com>
Co-authored-by: levoncrypto <95240473+levoncrypto@users.noreply.github.com>
Co-authored-by: levonpetrosyan93 <45027856+levonpetrosyan93@users.noreply.github.com>

* Export View Keys (#1543)

* Add an RPC command to export the Spark view key.

* Show Spark View Key in Qt.

* Sigma pool closed, Extra payload extended (#1477)

* Change of emission rules

* Fixes for testnet

* Cleaning up code and tests

* Workaround for current devnet bugs

* Workaround for testnet

* Devnet parameter tweak

* Sigma pool closed

* Extra payload size limit increased

* Changed HF block for testnet

* Initial spark name architecture

* Spark address ownership proofs implemented.

* Missing files added

* Check the ownership proof for spark names: initial implementation

* Fixes to the core part of spark names

* Added additional field (core)

* Consensus parameters for spark names

* Fixed mempool bug

* Fixes in spark name conflict resolution

* RPCs for spark names

* Additional API for spark names tx creation

* Changed way of checking spark name tx

* Wallet API for spark name transaction creation

* API changes for spark name tx creation

* Added registersparkname RPC call

* Spark activation check for RPC

* Make spark names case-insensitive

* Spark name RPC fixes

* Faster lookup for spark name by spark address

* Fixes for spark name/address lookup

* Improvements for duplicated address detection

* Fixes for spark name state

* Block index entries for spark names

* Make dot (.) a legit symbol in spark name

* Spark name block number for testnet

* Fixed restoration of old spark name data if block is disconnected

* API for quick check of spark name transaction validity before the creation

* added isAddressMine function

* Check if the address belongs to the wallet before creating spark name transaction

* Fixed fee calculation for spark name

* Fix for spark names RPC

* Added ability to spend to spark names in "sparkspend" RPC

* UI fixes

* Additional validations

* Fix for crash with spend to unregistered name

* Fixed fee value when registering spark name for more than one year

* Spark name UI improvements

* UI modifications for sending to spark name

* Address book fixes related to spark names

* Fixed period of validity when creating spark name through GUI

* Extended spark name error info for RPC

* Fixed crash on non-HD wallet

* Request wallet password for spark name creation

* Fixed fee calculation for the spark name tx

* Fixed detection of previously used spark address for a spark name

* Unit test for spark names

* Additional unit tests

* Fixes #1533

* Testnet HF block set

* Mainnet HF block set

---------

Co-authored-by: Peter Shugalev <peter@shugalev.com>
Co-authored-by: levoncrypto <levoncrypto1994@gmail.com>

* Build fix (#1553)

* Build fix

* coderabbitai comment resolved

* Duplicated rpc removed

* Bump version to v0.14.14.1 Spark Names (#1550)

* secp256k1: CMake build system added

* cmake: add cmake folder

* bench: Add initial cmake support

* Add initial main CMakeLists.txt

* doc: add initial cmake support

* univalue: add initial cmake support

* zmq: add initial cmake support

* crypto: add initial cmake support

* wallet: add initial cmake support

* src: initial add of src/CMakeLists.txt

* toolchain.cmake.in: Adding toolchain.cmake.in support

* crypto: add support for CMake function check.

* bitcoin-cli: add CMake compilation.

* firo-tx: add CMake compilation.
Improve miscellaneous CMake files.

* firod: add CMake compilation support.
    * Improved misc CMake compilation support.
    * Add bitcoin-config.h generation support.

* gmp: fix gmp link on OS X

* build: generate Linux installable package.

* CMake: add tests (test_bitcoin) compilation support.

* Qt: CMake GUI compilation support (Ubuntu)

* src/qt: Add macOS support for GUI with CMake compilation.

* depends: fix gmp compilation issue with mingw

* build: Add MingW support for CMake build system

* src: add backtrace compilation support macOS

* src: add backtrace compilation support Linux and MinGW-w64

* CMake: apply CodeRabbitAI suggestions.

* CMake: Add CI tasks for CMake build.

* Remove static from functions to fix debug linking

In release builds, these functions are inlined, avoiding linkage issues.
Debug builds (without inlining) caused undefined references in dependent
libraries due to static visibility. Removing static resolves this while
maintaining intended behavior in optimized builds.

* Add removed testcases and make BUILD_GUI=ON default option

* Added documentation to readme.md and fixed a lelantus test issue

* Fixed documentation, set ENABLE_WALLET=ON by default
Remove outdated old sigma testcase

* Rebase to Wed Apr 23 11:39:34 AM BST 2025

---------

Co-authored-by: levonpetrosyan93 <45027856+levonpetrosyan93@users.noreply.github.com>
Co-authored-by: firstcryptoman <firstcryptoman@gmail.com>
Co-authored-by: psolstice <peter@shugalev.com>
Co-authored-by: levonpetrosyan93 <petrosyan.levon93@gmail.com>
Co-authored-by: levoncrypto <levoncrypto1994@gmail.com>
Co-authored-by: levoncrypto <95240473+levoncrypto@users.noreply.github.com>
Co-authored-by: cassandras-lies <203535133+cassandras-lies@users.noreply.github.com>
Co-authored-by: justanwar <42809091+justanwar@users.noreply.github.com>
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