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

Lior/fix run tests #26

Closed
wants to merge 22 commits into from
Closed

Lior/fix run tests #26

wants to merge 22 commits into from

Conversation

liorgold2
Copy link
Contributor

@liorgold2 liorgold2 commented Jul 22, 2024

Pull Request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no API changes)
  • Build-related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this introduce a breaking change?

  • Yes
  • No

Other information


This change is Reviewable

dafnamatsry and others added 21 commits May 2, 2024 14:06
fix(format): run format fix on all repo
* feature(ci): Identify which package need to be built according to file changed
---------

Signed-off-by: Dori Medini <[email protected]>
Co-authored-by: Dori Medini <[email protected]>
* chore: fix ci to always compare changes against the target branch
* chore: copy all crates

* chore: switch to local dependencies

* chore: delete old papyrus_test_utils crate

* chore: rename crate test_utils -> papyrus_test_utils

* chore: fix RPCTransactio name in the mempool

* chore: fix mockito deps in papyrus and gateway

* chore: update cairo-* deps version

* chore: update config and presets for mempool and papyrus

* chore: rename default_config.json -> papyrus_default_config.json

* chore: rustfmt

* chore: reorg folders

* chore: reorg folders

* chore: fix unused deps

* chore: update papyrus Dockerfile

* chore: update scripts folder

* chore: copy build_native_blockifier.sh

* chore: copy BUILD and WORKSPACE files (from blockifier and committer)

* chore: update papyrus non crates folders and files

* chore: copy blockifier docs

* chore: copy committer taplo.toml

* chore: fix config tests

* chore: copy blockifier docker file

* fix: papyrus integration test runs only Papyrus related tests

* chore: rename conflicting dump_config.rs files

* chore: fix cargo doc errors

* chore: fix CI

* chore: small optimizations for the papyrus CI

* chore: copy README files

* chore: meld common repo files

---------

Co-authored-by: alon.dotan <[email protected]>
Co-authored-by: Dan Brownstein <[email protected]>
@liorgold2 liorgold2 requested a review from dan-starkware as a code owner July 22, 2024 12:54
@liorgold2 liorgold2 force-pushed the lior/fix_run_tests branch from 7f7d73d to f553d35 Compare July 22, 2024 12:59
@dorimedini-starkware dorimedini-starkware force-pushed the lior/fix_run_tests branch 5 times, most recently from 355310a to 7bdbcb5 Compare July 22, 2024 14:07
Copy link
Collaborator

@dorimedini-starkware dorimedini-starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewable status: 0 of 3 files reviewed, 1 unresolved discussion (waiting on @dan-starkware and @liorgold2)

a discussion (no related file):
made dummy change to lib.rs in the blockifier crate, and I see this in the run-tests phase, so it seems to work :)
image.png


@liorgold2 liorgold2 closed this Jul 24, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Jul 27, 2024
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.

5 participants