Skip to content
This repository has been archived by the owner on Nov 27, 2024. It is now read-only.

chore: update submodules #17

Merged
merged 5 commits into from
Dec 5, 2023
Merged

chore: update submodules #17

merged 5 commits into from
Dec 5, 2023

Conversation

AustinGreen
Copy link
Contributor

Motivation:

Removing harcoded dependencies and adding as submodules. These were copied over incorrectly during the repo migration.

Modifications:

Delete solmate and solady directories in lib directory.

Result:

We will reference our dependencies as submodules.

@AustinGreen AustinGreen merged commit db98fef into main Dec 5, 2023
2 of 4 checks passed
@AustinGreen AustinGreen deleted the austin/fix-submodules branch December 5, 2023 23:42
Copy link

github-actions bot commented Dec 5, 2023

Coverage after merging austin/fix-submodules into main will be

28.53%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src/lib
   ERC721NonTransferableMinimalProxy.sol0%0%0%0%102, 104, 106, 118–120, 128, 128, 128, 130, 130, 130, 134, 137, 139, 143, 145, 145, 145, 149, 152, 154, 156, 164, 166, 166, 166, 175, 177, 177, 177, 190, 41, 41, 41, 45, 45, 45, 47, 63–64, 72, 74, 74, 74, 76, 78, 82, 84, 88, 88, 88, 90, 90, 90, 92, 92, 92, 97, 99
   LlamaUtils.sol0%0%0%0%13, 13, 13–14, 19, 19, 19–20, 26, 32
   PolicyholderCheckpoints.sol0%0%0%0%101–102, 110, 123, 125, 125, 125, 127, 130, 130, 130, 133, 133, 133–136, 138, 140, 142–143, 160–162, 162, 162–163, 165, 168, 184–186, 186, 186–187, 189, 192, 202, 211, 222, 229, 229–231, 233, 233–235, 237, 237–239, 241, 241–243, 262, 265–271, 278, 38, 38, 38–39, 41, 43–44, 46, 46, 46–48, 48, 48–49, 51, 55, 57, 72, 79–80, 97–98, 98, 98–99
   SupplyCheckpoints.sol0%0%0%0%100–101, 109, 124, 126, 126, 126, 128, 131, 131, 131, 134, 134, 134–137, 139, 142, 159–161, 161, 161–162, 164, 167, 183–185, 185, 185–186, 188, 191, 201, 210, 213, 215–216, 224, 235, 242, 242–244, 246, 246–248, 250, 250–252, 254, 254–256, 275, 278–284, 291, 42, 42, 42–43, 45, 47–48, 50, 50, 50–52, 52, 52–53, 55, 59, 61, 71, 78–79, 96–97, 97, 97–98
src/token-voting
   ERC20TokenholderActionCreator.sol100%100%100%100%
   ERC20TokenholderCaster.sol100%100%100%100%
   ERC721TokenholderActionCreator.sol0%100%0%0%27, 31, 35
   ERC721TokenholderCaster.sol0%100%0%0%24, 28, 32
   LlamaTokenVotingFactory.sol0%0%0%0%47–48, 48, 48–50, 52–53, 65–66, 73–74, 84–85, 95–96
   TokenholderActionCreator.sol47.76%64.29%35.29%47.22%154–156, 156, 156–157, 177–179, 179, 179–180, 197, 211–212, 241–242, 249, 269, 271, 285, 294, 298, 303
   TokenholderCaster.sol76.74%85%38.89%78.72%223–225, 225, 225–226, 250–252, 252, 252–253, 273–274, 307–308, 355–356, 380–381, 392, 402–403, 410, 425, 436, 447, 458, 463

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant