-
Notifications
You must be signed in to change notification settings - Fork 2
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 project and contracts to SablierFlow #120
Conversation
ce16e76
to
7c94173
Compare
docs: update README.md docs: capitalize flow to Flow test: fix misspelled labelContracts in Bast.t.sol
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two comments:
- We should always use Capitalized version in the docs when referring to "Flow" instead of "flow" because its a name.
- Since flow is also a verb, some of the docs stopped making sense which I have addressed in .
Adddressed in ceb9b0e.
test: rename deploy contract functions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lezzz go with Flow 🚀
haha 😂 @smol-ninja pushed a new commit, lmk if it looks good |
I have seen it. Good to go. |
Linked Issues
SablierV2OpenEnded
toSablierFlow
#95.Changelog
Note: after this PR is merged, we also need to update the repo name.