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.

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

Fix some comments #99

Merged
merged 3 commits into from
Feb 5, 2024
Merged

Conversation

jordibonet-lambdaclass
Copy link

What ❔

Why ❔

Checklist

  • PR title corresponds to the body of PR (we generate changelog entries from PRs).
  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.
  • Code has been formatted via zk fmt and zk lint.
  • Spellcheck has been run via zk spellcheck.
  • Linkcheck has been run via zk linkcheck.

@toni-calvin toni-calvin merged commit 400d355 into fix-integration-test Feb 5, 2024
4 of 8 checks passed
@toni-calvin toni-calvin deleted the validium-tests-fix-comments branch February 5, 2024 11:55
ilitteri pushed a commit that referenced this pull request Feb 15, 2024
* Fix eth_sender value

* Restore original config file and omit commented values

* Format
mationorato pushed a commit that referenced this pull request Feb 15, 2024
* abstract setup mode method

* remove blank line

* add config mode in all init methods

* update log

* Fix some comments (#99)

* Fix eth_sender value

* Restore original config file and omit commented values

* Format

* abstract methods and constants

* remove async

* remove unnecessary async

* change constant value

* add cosntant

* fix: test new vars (#114)

* Update new variables between modes

* Format

* Fix toml format

* Fix remove line

* fix bug where dev.env is not overwritten

* Revert "fix bug where dev.env is not overwritten"

This reverts commit 051f173.

* fix bug in dev.env load

* Fix updateConfigFile

* Fix initialization

* Remove repeated constant definitions

* Remove internal enforced l1 gas price set

---------

Co-authored-by: toni-calvin <antonio.calvin@lambdaclass.com>
Co-authored-by: Jordi <146965181+jordibonet-lambdaclass@users.noreply.github.com>
Co-authored-by: Jordi <jordi.bonet.valiente@lambdaclass.com>
Co-authored-by: Joaquin Carletti <joaquin.carletti@lambdaclass.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants