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

Replace while loop with for loop once references are allowed #385

Open
SupunS opened this issue Sep 22, 2023 · 0 comments
Open

Replace while loop with for loop once references are allowed #385

SupunS opened this issue Sep 22, 2023 · 0 comments
Assignees
@SupunS
Labels
SC-Eng Issues that we want to see surfaced in SC-Eng ZH Board Technical Debt

Comments

@SupunS
Copy link
Member

SupunS commented Sep 22, 2023

Issue To Be Solved

As discussed in #382 (comment), The code had to be rewritten using a while loop, because references are not allowed in the for-loop.

Suggest A Solution

Revert this change once references are allowed to be used in for loop: onflow/cadence#2784
@SupunS SupunS self-assigned this Sep 22, 2023
@SupunS SupunS mentioned this issue Sep 22, 2023
Merged
@joshuahannan joshuahannan added the SC-Eng Issues that we want to see surfaced in SC-Eng ZH Board label Sep 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SupunS SupunS

Labels
SC-Eng Issues that we want to see surfaced in SC-Eng ZH Board Technical Debt
Projects
🌊 Flow 4D
Status: No status
Milestone
No milestone
Development

No branches or pull requests
2 participants
@joshuahannan @SupunS