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

Revert Alias of Multi-Index Operations #262

Merged
merged 4 commits into from
Jan 17, 2024

Conversation

ericpassmore
Copy link
Contributor

@ericpassmore ericpassmore commented Dec 21, 2023

Revert the following commits from main branch due to lack of testing, and not in any release. See issue #260 for issue and PR history.

This update is the produce of the following command.
git revert 65f928f a1aae5f 2b0fd5c 94ca050

Validated by comparing against pre-merge commit.
git diff e77c18e74 libraries/eosiolib/contracts/eosio/multi_index.hpp
git diff e77c18e74 tests/unit/test_contracts/

Running Build and Test on Branch Now

@ericpassmore ericpassmore self-assigned this Dec 21, 2023
@ericpassmore ericpassmore linked an issue Dec 21, 2023 that may be closed by this pull request
@ericpassmore
Copy link
Contributor Author

ericpassmore commented Dec 23, 2023

Note:start
group: CLEANCODE
category: INTERNALS
summary: revert multi-index alias operations
Note:end

@ericpassmore ericpassmore merged commit 6906563 into main Jan 17, 2024
12 checks passed
@ericpassmore ericpassmore deleted the ehp/gh-260-revert-db-index-op-names branch January 17, 2024 01:47
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.

Revert insert, remove and update methods to multi_index
4 participants