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

Support --no-indent and --visualize flags #1779

Merged
merged 4 commits into from
May 11, 2023
Merged

Support --no-indent and --visualize flags #1779

merged 4 commits into from
May 11, 2023

Conversation

Shaikh-Ubaid
Copy link
Collaborator

@Shaikh-Ubaid Shaikh-Ubaid requested a review from certik May 11, 2023 08:08
@Shaikh-Ubaid Shaikh-Ubaid force-pushed the visualize_and_indent branch 2 times, most recently from 0e2a049 to eaf38be Compare May 11, 2023 08:13
@Shaikh-Ubaid
Copy link
Collaborator Author

Ready.

@Shaikh-Ubaid Shaikh-Ubaid changed the title Default indent and visualize Support --no-indent and --visualize flags May 11, 2023
@Shaikh-Ubaid
Copy link
Collaborator Author

The reverted commit Revert "TEST: Indent all AST and ASR tests" is present in main branch.

@certik
Copy link
Contributor

certik commented May 11, 2023

Thanks! Can you please resolve the conflict? I merged the other PR first. After that we can merge this.

@Shaikh-Ubaid
Copy link
Collaborator Author

Ready.

@certik certik merged commit 8c71d37 into lcompilers:main May 11, 2023
5 checks passed
@certik
Copy link
Contributor

certik commented May 11, 2023

Awesome, thank you for this!

@Shaikh-Ubaid Shaikh-Ubaid deleted the visualize_and_indent branch May 11, 2023 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@certik certik Awaiting requested review from certik

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Shaikh-Ubaid @certik