Skip to content

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

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

Merged
merged 4 commits into from
May 11, 2023

Conversation

ubaidsk
Copy link
Collaborator

@ubaidsk ubaidsk commented May 11, 2023

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

ubaidsk commented May 11, 2023

Ready.

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

ubaidsk commented May 11, 2023

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.

@ubaidsk ubaidsk force-pushed the visualize_and_indent branch from eaf38be to 8d1c9c3 Compare May 11, 2023 14:39
@ubaidsk
Copy link
Collaborator Author

ubaidsk commented May 11, 2023

Ready.

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

certik commented May 11, 2023

Awesome, thank you for this!

@ubaidsk ubaidsk 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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants