chore: remove integration tests #196
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
cargo install will always clone all submodules, which is bad ux rust-lang/cargo#4247
shekhirin
added a commit
to shekhirin/foundry
that referenced
this pull request
Dec 17, 2021
This reverts commit bab7e2d.
gakonst
pushed a commit
that referenced
this pull request
Dec 17, 2021
* Revert "chore: remove integration tests (#196)" This reverts commit bab7e2d. * add Makefile, prepare CI matrix dynamically * fix set-matrix command * add install libusb step in CI for integration tests * clone testdata recursively * update README.md and improve Makefile * add missing depth and recursive git flags * better Makefile formatting * use u128 in fuzz/mean_gas * use release build * run e2e on master push only
shekhirin
added a commit
to shekhirin/foundry
that referenced
this pull request
Dec 18, 2021
This reverts commit bab7e2d.
gakonst
pushed a commit
that referenced
this pull request
Feb 14, 2022
* feat: add fmt pallet * feat: add formatter skeleton * feat(fmt): Visitable for source & contract parts * dyn -> impl, contract & enum impls, tests * add empty_brackets helper * use indent_write, implement more visitors * cleanup test_formatter helper * better brackets helpers * add multiline support and respect to line_length * fix clippy * remove itertools dep * add llvm to ci tests * add llvm to env var * add llvm install and env var set to lint step * respect basic style guide * fix Cargo.lock * use solang without LLVM * update solang, remove LLVM from CI * bump solang * final solang bump * add README.md, improve comments * fix README.md * test against prettier-plugin-solidity snapshots * add submodules recursively in CI * Revert "chore: remove integration tests (#196)" This reverts commit bab7e2d. * add fmt testdata to Makefile * run make fmt-testdata in CI * cli & better formatting * fix constructor function corner case * use separate solang-parser crate * add --check option * bump solang, fix missing parts regarding source visiting * bump solang, refactor blank lines formatting * fix semver parsing (remove it lol) * improve cli args * parallel formatting w/ rayon, improve errors * fix Makefile fmt-testdata dir structure * fixes after matt's review * good looking diffs on --check * fix lint * more default Visitor implementations * bump solang, improve doc comments * fix clippy Signed-off-by: Alexey Shekhirin <a.shekhirin@gmail.com> * bump solang Signed-off-by: Alexey Shekhirin <a.shekhirin@gmail.com> * comment out fmt from cli * support error definitions & fix clippy Co-authored-by: Matthias Seitz <matthias.seitz@outlook.de>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
cargo install will always clone all submodules, which is bad ux
https://github.com/rust-lang/cargo/issues/4247\#issuecomment-959912231