Skip to content

LinkParsing: Support "foo line 123"#175906

Merged
Tyriar merged 1 commit into
microsoft:mainfrom
orgads:terraform-links
Mar 6, 2023
Merged

LinkParsing: Support "foo line 123"#175906
Tyriar merged 1 commit into
microsoft:mainfrom
orgads:terraform-links

Conversation

@orgads
Copy link
Copy Markdown
Contributor

@orgads orgads commented Mar 2, 2023

Terraform output.

Example:

│ Error: Invalid reference
│
│   on main.tf line 2, in locals:
│    2:   foo = abcd

Fixes #175896

Terraform output.

Example:
│ Error: Invalid reference
│
│   on main.tf line 2, in locals:
│    2:   foo = abcd

Fixes microsoft#175896
@orgads orgads force-pushed the terraform-links branch from 48b5701 to 80ceaeb Compare March 6, 2023 07:00
@orgads
Copy link
Copy Markdown
Contributor Author

orgads commented Mar 6, 2023

@Tyriar Can you please review?

Copy link
Copy Markdown
Contributor

@Tyriar Tyriar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks good 👍

@Tyriar Tyriar merged commit 6f20c42 into microsoft:main Mar 6, 2023
@orgads orgads deleted the terraform-links branch March 6, 2023 15:27
@orgads
Copy link
Copy Markdown
Contributor Author

orgads commented Mar 6, 2023

cc @yoshigev

@github-actions github-actions Bot locked and limited conversation to collaborators Apr 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Terraform error format is not covered by error renderer

4 participants