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.

Sign up for GitHub

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

Jump to bottom

fix: Use shorter requires (C4 #3) #16

Merged

deluca-mike merged 1 commit into main from sc-4271-shorter-requires

Dec 11, 2021

lucas-manuel commented Dec 10, 2021

Description

Addresses code-423n4/2021-12-maple-findings#3

Integrations Checklist

Have any function signatures changed? If yes, outline below.
Have any features changed or been added? If yes, outline below.
Have any events changed or been added? If yes, outline below.
Has all documentation been updated?

Changelog

Function Signature Changes

Features

Events


          fix: use shorter requires

60acd5f

shortcut-integration bot commented Dec 10, 2021

This pull request has been linked to Shortcut Story #4271: [c4] Issue #3 - Require messages.

lucas-manuel self-assigned this

lucas-manuel requested review from deluca-mike and JGcarv

December 10, 2021 13:11

lucas-manuel added Priority: High Priority: Medium Status: Ready for Review and removed Priority: Medium labels

deluca-mike approved these changes

View reviewed changes

deluca-mike merged commit 0e36cbe into main

deluca-mike deleted the sc-4271-shorter-requires branch

December 11, 2021 22:59

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority: High Status: Ready for Review