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

Release 1.6.4 #285

Merged
merged 9 commits into from
Sep 29, 2021
Merged

Release 1.6.4 #285

merged 9 commits into from
Sep 29, 2021

Conversation

birchmd
Copy link
Member

@birchmd birchmd commented Sep 24, 2021

birchmd and others added 6 commits September 24, 2021 18:19
* A test showing a 1inch liquidity pool working

* Add some modules to organize 1inch code

* Reorganize code; add gas value asserts

* Avoid some code duplication
#275)

Bumps [@openzeppelin/contracts](https://github.com/OpenZeppelin/openzeppelin-contracts) from 4.3.1 to 4.3.2.
- [Release notes](https://github.com/OpenZeppelin/openzeppelin-contracts/releases)
- [Changelog](https://github.com/OpenZeppelin/openzeppelin-contracts/blob/master/CHANGELOG.md)
- [Commits](OpenZeppelin/openzeppelin-contracts@v4.3.1...v4.3.2)

---
updated-dependencies:
- dependency-name: "@openzeppelin/contracts"
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Introduce workspace
* Split to crate:
  *  engine
  * prelude
  * sdk
* Fixed tests
* Refactored dependencies to new crates

Co-authored-by: Michael Birch <michael.birch@aurora.dev>
@mrLSD
Copy link
Member

mrLSD commented Sep 24, 2021

@birchmd From a source code perspective, it's can be version 1.6.4. It should be 1.7.0 - because of breaking changes in source code.

@birchmd
Copy link
Member Author

birchmd commented Sep 24, 2021

As I understand it, our product is the (wasm) contract we deploy to Near and this is what the version refers to. Source code organization changes (even large ones like spitting into multiple crates) are not observable from the contract perspective. The only important change is the ft_metadata bug fix, which is why I chose 1.6.4 as the version.

Copy link
Contributor

@joshuajbouw joshuajbouw left a comment

Choose a reason for hiding this comment

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

This looks good to me. There are some small nits that I do notice such as the libs IMO shouldn't be prefixed with engine- but, that's ok for now. Everything else and running it on my end looks good.

To be fair, we should be bumping every major version every time we break public API. We are still technically beta, although the version says otherwise. Bumping minor, in this case, should be fine then.

But, if the API isn't correct and producing something unintentionally, that can be considered a bug.

@mrLSD mrLSD self-requested a review September 27, 2021 07:43
@artob artob added the C-release Category: New release. label Sep 29, 2021
@artob
Copy link
Contributor

artob commented Sep 29, 2021

[O]ur product is the (wasm) contract we deploy to Near and this is what the version refers to [...]
We should be bumping every major version every time we break public API.

Correct. We follow semantic versioning, and the versioning scheme is quite systematic and predictable.

@artob artob merged commit 008e34c into master Sep 29, 2021
@artob artob deleted the release-1.6.4 branch September 29, 2021 15:53
artob added a commit that referenced this pull request Sep 29, 2021
artob added a commit that referenced this pull request Sep 29, 2021
* Split to crates: engine, prelude, sdk (#243)
* Test for 1inch liquidity protocol (#274)
* Bump @openzeppelin/contracts from 4.3.1 to 4.3.2 in /etc/eth-contracts (#275)
* Fix deploy benchmark to use a realistic transaction (that actually writes data to the created account code) (#277)
* Fix json formatting in JsonValue serialization (#279)
* Update primitive-types dependency to 0.10 (#280)
* Fix block.timestamp (#287)

Co-authored-by: Evgeny Ukhanov <evgeny@aurora.dev>
Co-authored-by: Michael Birch <michael@aurora.dev>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-release Category: New release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants