Skip to content

[fix][doc] explicitly instruct clone the repo#69

Merged
BewareMyPower merged 1 commit intoapache:mainfrom
tisonkun:patch-1
Oct 25, 2022
Merged

[fix][doc] explicitly instruct clone the repo#69
BewareMyPower merged 1 commit intoapache:mainfrom
tisonkun:patch-1

Conversation

@tisonkun
Copy link
Member

Documentation

  • doc-required
    (Your PR needs to update docs and you will update later)

  • doc-not-needed
    (Please explain why)

  • doc
    (Your PR contains doc changes)

  • doc-complete
    (Docs have been already added)


- [Requirements](#requirements)
- [Platforms](#platforms)
- [Compilation](#compilation)
Copy link
Member

Choose a reason for hiding this comment

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

Why remove the catalog?

Copy link
Member Author

Choose a reason for hiding this comment

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

@tisonkun
Copy link
Member Author

Shall we merge this patch now? Or we're still waiting for specific reviewers' comment?

@BewareMyPower
Copy link
Contributor

No. Just because it's not an urgent PR. It's ready to merge.

@BewareMyPower BewareMyPower merged commit ec9271d into apache:main Oct 25, 2022
@tisonkun tisonkun deleted the patch-1 branch October 25, 2022 14:35
BewareMyPower pushed a commit that referenced this pull request Nov 1, 2022
@RobertIndie RobertIndie added this to the 3.1.0 milestone Nov 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants