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

Sync feature/string-templates branch with master #3657

Merged
merged 76 commits into from
Oct 30, 2024
Merged

Sync feature/string-templates branch with master #3657

merged 76 commits into from
Oct 30, 2024

Conversation

SupunS
Copy link
Member

@SupunS SupunS commented Oct 30, 2024

  • Targeted PR against master branch
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work
  • Code follows the standards mentioned here
  • Updated relevant documentation
  • Re-reviewed Files changed in the Github PR explorer
  • Added appropriate labels

RZhang05 and others added 30 commits October 16, 2024 16:49
@RZhang05
Remove checking and parsing from suite, fix windows issues.
298fa4d
@RZhang05
Update flow-go version to pass tests.
8ce8986
@RZhang05
Update to latest cadence version.
314bca6
@RZhang05
Fix formatting, add versioning back.
8800bfe
@turbolent
remove Crypto contract
2874423
@turbolent
call constructor for contracts loaded from identifier location, test
393e955
@turbolent
remove Flow CLI and testing from CI job
bc51dde
@RZhang05
Remove unused code, always use latest flow-go version.
4f3e6fa
@SupunS
Merge branch 'master' of https://github.com/onflow/cadence into basti…
2d0b760 
…an/3135-move-crypto-contract-onchain
@SupunS
Merge branch 'master' of https://github.com/onflow/cadence into basti…
7253dcd 
…an/3135-move-crypto-contract-onchain
@turbolent
fix dependencies / breakage due to restructuring
2fcd4c5
@turbolent
Merge branch 'master' into source-compatibility-update
0ffd972
@turbolent
remove All function again, export field instead
02b9413
@RZhang05
Remove unused files property.
8805722
@RZhang05
Merge branch 'source-compatibility-update' of https://github.com/RZha…
69bd725 
…ng05/cadence into source-compatibility-update
@turbolent
Merge pull request #3616 from RZhang05/source-compatibility-update
bdf0912
@turbolent
revert 393e955: use resolve location to rewrite import
2e72919
@turbolent
Merge pull request #3627 from onflow/bastian/fix-storage-explorer-2
d2f3abc
@turbolent
bring back license header
009255f
@SupunS
v1.1.0
db075f7
@turbolent
Merge pull request #3628 from onflow/release/v1.1.0
0b1fc2b
@SupunS
Update release docs with a section for versioning strategy
bd46d4f
@turbolent
Merge branch 'master' into bastian/3135-move-crypto-contract-onchain
7f1b6fb
@SupunS
Cleanup outdated FT/NFT related contract codes and tests
cf0472b
@turbolent
Merge pull request #3619 from onflow/bastian/3135-move-crypto-contrac…
df33ab8 
…t-onchain
@SupunS
Fix contracts backward compatibility checker
f52a7ed
@SupunS
Merge pull request #3630 from onflow/supun/cleanup
d96a90d 
Cleanup outdated FT/NFT related contract codes and tests
@turbolent
re-enable and update the Source Compatiblity Suite GitHub action
2ee8c64
@turbolent
update email addresses
e020579
@turbolent
v1.2.0
4934d4a
franklywatson and others added 21 commits October 25, 2024 21:35
@franklywatson
Update index.tsx
aa3809d
@turbolent
Merge pull request #3646 from onflow/jp-c1.0-mop-up
0b92557
@turbolent
update old uses of pub and priv
aceb884
@turbolent
document Go patterns used in the codebase
a0ef085
@turbolent
move architecture diagram to contributor documentation directory
52bbfa4
@turbolent
move runtime tests
dccae9a
@turbolent
fix test expectation
08eae83
@turbolent
remove outdated gitattributes for old parser
ed36b6c
@turbolent
move checker tests to sema package
258b177
@turbolent
move occurrence matcher
0f5d513
@turbolent
move interpreter tests
a6da1a0
@turbolent
bring back access tests
bc3bc62
@turbolent
refactor test utils
782db6c
@turbolent
adjust paths
ff8d29d
@turbolent
Merge pull request #3651 from onflow/bastian/refactor-tests
e20bd56
@turbolent
Merge pull request #3650 from onflow/bastian/move-architecture-diagram
0f382af
@turbolent
Merge pull request #3649 from onflow/bastian/document-go-patterns
1c029f3
@turbolent
Merge pull request #3652 from onflow/bastian/refactor-test-utils
63d3591
@turbolent
Merge pull request #3647 from onflow/bastian/pub-priv-cleanup
335ace9
@turbolent
Merge pull request #3644 from onflow/bastian/compat-suite-cadence-tests
1399a6d
@SupunS
Merge branch 'master' of https://github.com/onflow/cadence into featu…
f3b4828 
…re/string-templates
@SupunS SupunS self-assigned this Oct 30, 2024
@github-actions GitHub Actions
Copy link

Cadence Benchstat comparison

This branch with compared with the base branch onflow:feature/string-templates commit a185a42
The command for i in {1..N}; do go test ./... -run=XXX -bench=. -benchmem -shuffle=on; done was used.
Bench tests were run a total of 7 times on each branch.

Collapsed results for better readability

@SupunS SupunS marked this pull request as ready for review October 30, 2024 19:14
@SupunS SupunS merged commit 4f9e343 into feature/string-templates Oct 30, 2024
7 of 9 checks passed
@SupunS SupunS deleted the supun/update-string-templates branch October 30, 2024 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@turbolent turbolent turbolent approved these changes

Assignees

@SupunS SupunS

Labels
Technical Debt
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@SupunS @turbolent @RZhang05 @fxamacker @franklywatson