Skip to content

feat[lang]!: remove deprecated bitwise builtins#4552

Merged
charles-cooper merged 3 commits intovyperlang:masterfrom
cyberthirst:feat/remove-bitwise-builtins
Apr 5, 2025
Merged

feat[lang]!: remove deprecated bitwise builtins#4552
charles-cooper merged 3 commits intovyperlang:masterfrom
cyberthirst:feat/remove-bitwise-builtins

Conversation

@cyberthirst
Copy link
Copy Markdown
Collaborator

@cyberthirst
Copy link
Copy Markdown
Collaborator Author

maybe we should move some of the docs to the operator part

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 2, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.56%. Comparing base (cef84e8) to head (1f2aba5).
Report is 94 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4552      +/-   ##
==========================================
+ Coverage   92.35%   92.56%   +0.21%     
==========================================
  Files         123      123              
  Lines       17543    17467      -76     
  Branches     2961     2950      -11     
==========================================
- Hits        16201    16169      -32     
+ Misses        938      894      -44     
  Partials      404      404              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Bitwise Operations
==================

.. py:function:: bitwise_and(x: uint256, y: uint256) -> uint256
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

we should add a comment like

.. note::
    The functions ``bitwise_and``, ``bitwise_or``, ``bitwise_xor`` and ``bitwise_not`` have been removed in Vyper 0.4.2. Please use their operator versions instead: ``&``, ``|``, ``^``, ``~``.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

maybe also add that it was deprecated since 0.3.4

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

added a note: 1f2aba5

Copy link
Copy Markdown
Member

@charles-cooper charles-cooper left a comment

Choose a reason for hiding this comment

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

@charles-cooper charles-cooper merged commit d3458d4 into vyperlang:master Apr 5, 2025
161 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