Skip to content

Commit

Permalink
maybe fix coverage
Browse files Browse the repository at this point in the history
  • Loading branch information
StanislavBreadless committed Sep 20, 2024
1 parent 9a92955 commit e32f0bd
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/l1-contracts-ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -286,7 +286,7 @@ jobs:
- name: Filter directories
run: |
sudo apt update && sudo apt install -y lcov
lcov --remove lcov.info 'test/*' 'contracts/dev-contracts/*' '../lib/forge-std/*' '../lib/murky/*' 'lib/*' '../lib/*' 'lib/' --output-file lcov.info --rc lcov_branch_coverage=1
lcov --remove lcov.info 'test/*' 'contracts/dev-contracts/*' '../lib/forge-std/*' '../lib/murky/*' 'lib/*' '../lib/*' 'lib/' 'deploy-scripts/*' --output-file lcov.info --rc lcov_branch_coverage=1
# This step posts a detailed coverage report as a comment and deletes previous comments on
# each push. The below step is used to fail coverage if the specified coverage threshold is
Expand Down
2 changes: 1 addition & 1 deletion l1-contracts/deploy-scripts/DeployL2Contracts.sol
Original file line number Diff line number Diff line change
Expand Up @@ -104,10 +104,10 @@ contract DeployL2Script is Script {

function saveOutput() internal {
vm.serializeAddress("root", "l2_da_validator_address", deployed.l2DaValidatorAddress);
string memory toml = vm.serializeAddress("root", "l2_default_upgrader", deployed.forceDeployUpgraderAddress);

vm.serializeAddress("root", "consensus_registry_implementation", deployed.consensusRegistryImplementation);
vm.serializeAddress("root", "consensus_registry_proxy", deployed.consensusRegistryProxy);
string memory toml = vm.serializeAddress("root", "l2_default_upgrader", deployed.forceDeployUpgraderAddress);

string memory root = vm.projectRoot();
string memory path = string.concat(root, "/script-out/output-deploy-l2-contracts.toml");
Expand Down

0 comments on commit e32f0bd

Please sign in to comment.