Skip to content
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

Feature/feijoa #227

Draft
wants to merge 76 commits into
base: develop
Choose a base branch
from
Draft

Feature/feijoa #227

wants to merge 76 commits into from
+39,889 −50,717

Conversation

invocamanman
Copy link
Collaborator

No description provided.

@cla-bot cla-bot bot added the cla-signed label Mar 28, 2024
@invocamanman invocamanman marked this pull request as ready for review April 8, 2024 14:05
simonDos and others added 26 commits April 11, 2024 17:18
@simonDos
work on PolygonRollupBaseFeijoa coverage
b185c63
@DhairyaSethi
tests: global exit root v2 after initalization
2ed6a94
@DhairyaSethi
tests: global exit root v2 before initialization
cd4b644
@DhairyaSethi
test: updateExitRoot idempotentcy for coverage
5e86e19
@DhairyaSethi
test: bump coverage to 100
44fbf09
@DhairyaSethi
test: depositCount exceeds capacity
2984b63
@invocamanman
Merge pull request #235 from 0xPolygonHermez/test/GERv2
8aeae8f 
tests: global exit root v2
@invocamanman
Merge pull request #234 from 0xPolygonHermez/feijoaTests
dff1895 
work on PolygonRollupBaseFeijoa coverage
@ZeroEkkusu
test(feijoa): complete committee coverage
7430a1d
@invocamanman
Merge pull request #237 from ZeroEkkusu/test/coverage
784bdd3 
test(feijoa): complete committee coverage
@invocamanman
update l1 event
6d7957e
@invocamanman
Merge pull request #238 from 0xPolygonHermez/feature/updateL1Event
eb2f929 
update l1 event
@invocamanman
update tx btye constnat
131a930
@invocamanman
Merge pull request #240 from 0xPolygonHermez/feature/updateTxByteCons…
112cf3e 
…tant

update tx btye constnat
@DhairyaSethi
test(rollupmanager); coverage
6ba4086
@DhairyaSethi
test: coverage
dc2f102
@DhairyaSethi
test: use GERv2 for bridge tests
0bc1388
@DhairyaSethi
test: rebase l1event info changes
bb44e23
@DhairyaSethi
test: cleanup
7687547
@invocamanman
Merge pull request #236 from 0xPolygonHermez/tests/bridge
9311185 
use GERv2 for bridge tests
@invocamanman
Merge pull request #241 from 0xPolygonHermez/test/rollupmanager
18eef1e 
RollupManager increase coverage
@invocamanman
fix test to work with custom mnemonics
868e790
@ZeroEkkusu
test(validium): cover timestamp and leaf reverts
44d7ccf
@ZeroEkkusu
Merge pull request #242 from 0xPolygonHermez/test/coverage-val
fd65d86 
test(validium): cover timestamp and leaf reverts
@DhairyaSethi
Update CI (#243)
203dc37 
* chore: update ci to node 18
@invocamanman
config fixes
8d9ed46
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Apr 18, 2024

Quality Gate Passed Quality Gate passed

Issues
128 New issues
0 Accepted issues

Measures
12 Security Hotspots
No data about Coverage
52.0% Duplication on New Code

See analysis details on SonarCloud

@invocamanman invocamanman marked this pull request as draft June 11, 2024 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@krlosMata krlosMata Awaiting requested review from krlosMata krlosMata will be requested when the pull request is marked ready for review krlosMata is a code owner

@ignasirv ignasirv Awaiting requested review from ignasirv ignasirv will be requested when the pull request is marked ready for review ignasirv is a code owner

@laisolizq laisolizq Awaiting requested review from laisolizq laisolizq will be requested when the pull request is marked ready for review laisolizq is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
cla-signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@invocamanman @simonDos @DhairyaSethi @ZeroEkkusu