Skip to content

♻️ Use Keyword Arguments for Event Instantiation#280

Merged
pcaversaccio merged 6 commits intomainfrom
refactor/kwarg-events
Oct 7, 2024
Merged

♻️ Use Keyword Arguments for Event Instantiation#280
pcaversaccio merged 6 commits intomainfrom
refactor/kwarg-events

Conversation

@pcaversaccio
Copy link
Copy Markdown
Owner

@pcaversaccio pcaversaccio commented Oct 7, 2024

🕓 Changelog

Vyper changed event instantiation from taking positional arguments to taking keyword arguments (see PR #4257). This PR refactors all 🐍 snekmate event instantiations accordingly (resolves #277). Furthermore, we configure the tests to use the latest commit from the master branch of Vyper going forward, ensuring we are always testing against the most up-to-date version of the compiler.

🐶 Cute Animal Picture

image

@pcaversaccio pcaversaccio self-assigned this Oct 7, 2024
@pcaversaccio pcaversaccio added the refactor/cleanup ♻️ Code refactorings and cleanups label Oct 7, 2024
@pcaversaccio pcaversaccio added this to the 0.1.1 milestone Oct 7, 2024
@pcaversaccio pcaversaccio marked this pull request as draft October 7, 2024 10:33
Signed-off-by: Pascal Marco Caversaccio <[email protected]>
@pcaversaccio pcaversaccio added the documentation 📖 Improvements or additions to documentation label Oct 7, 2024
Signed-off-by: Pascal Marco Caversaccio <[email protected]>
Signed-off-by: Pascal Marco Caversaccio <[email protected]>
Signed-off-by: Pascal Marco Caversaccio <[email protected]>
Signed-off-by: Pascal Marco Caversaccio <[email protected]>
@pcaversaccio pcaversaccio marked this pull request as ready for review October 7, 2024 13:43
@pcaversaccio pcaversaccio merged commit ceb76b5 into main Oct 7, 2024
@pcaversaccio pcaversaccio deleted the refactor/kwarg-events branch October 7, 2024 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation 📖 Improvements or additions to documentation refactor/cleanup ♻️ Code refactorings and cleanups

Projects

None yet

Development

Successfully merging this pull request may close these issues.

♻️ Use Keyword Arguments for Event Instantiation

1 participant