Skip to content

Fix example link in gazelle readme - #693

Merged
f0rmiga merged 1 commit into
bazel-contrib:mainfrom
Whoaa512:patch-1
May 3, 2022
Merged

Fix example link in gazelle readme#693
f0rmiga merged 1 commit into
bazel-contrib:mainfrom
Whoaa512:patch-1

Conversation

@Whoaa512

Copy link
Copy Markdown
Contributor

PR Checklist

  • Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

docs fix

  • Documentation content changes

Other information

Use a permalink instead of relative file path

@Whoaa512
Whoaa512 requested a review from f0rmiga as a code owner April 28, 2022 22:36

@f0rmiga f0rmiga left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It doesn't make sense to use a permanent link since it creates a maintenance burden that doesn't fit our workflow. I added a proper fix as a suggestion.

Comment thread gazelle/README.md Outdated
Use a permalink instead of relative file path
@f0rmiga
f0rmiga merged commit 6948791 into bazel-contrib:main May 3, 2022
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