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

refactor: Rename wallet to account #226

Merged
merged 10 commits into from
Mar 3, 2022
Merged

Conversation

wal33d006
Copy link
Contributor

Big PR, nothing fancy just renamed everything that was possible.

@wal33d006 wal33d006 added Technical 🛠 Technical task not closely coupled with any feature 🛠 Refactor 🛠 labels Mar 3, 2022
@wal33d006 wal33d006 changed the title Refactor: Rename wallet to account refactor: Rename wallet to account Mar 3, 2022
Copy link
Contributor

@Zfinix Zfinix left a comment

Choose a reason for hiding this comment

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

LGTM 🙌🏾

Copy link
Contributor

@andrzejchm andrzejchm left a comment

Choose a reason for hiding this comment

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

good job! Since this is a public package used by other developers, we need to provide a migration guide for them, let's add a changelog file with small section of migration guide for versions older than 2.0.3 stating all the classes that have changed their names

@wal33d006
Copy link
Contributor Author

Updated CHANGELOG.md file.
Followed this convention (open to suggestions if that's not a very cool way).

Added previous change logs by simply picking up points from releases, most of which are not related to this package. Let me know if we need to remove them and add something else entirely.

@andrzejchm

Copy link
Contributor

@andrzejchm andrzejchm left a comment

Choose a reason for hiding this comment

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

@wal33d006 great work, thanks for taking care of the changelog!

@wal33d006 wal33d006 merged commit 42e3328 into main Mar 3, 2022
@wal33d006 wal33d006 deleted the refactor_wallet_to_account branch March 3, 2022 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠 Refactor 🛠 Technical 🛠 Technical task not closely coupled with any feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants