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.

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

docs: update plugin list #958

Merged
merged 2 commits into from
Aug 8, 2022
Merged

Conversation

aFlyBird0
Copy link
Member

@aFlyBird0 aFlyBird0 commented Aug 6, 2022

Signed-off-by: Bird aflybird0@gmail.com

Pre-Checklist

Note: please complete ALL items in the following checklist.

  • I have read through the CONTRIBUTING.md documentation.
  • My code has the necessary comments and documentation (if needed).
  • I have added relevant tests

Changes

image

image

image

Signed-off-by: Bird <aflybird0@gmail.com>
@aFlyBird0 aFlyBird0 requested a review from IronCore864 as a code owner August 6, 2022 11:06
@@ -133,8 +133,8 @@ tools:
# release name of the jenkins helm chart, mandatory
releaseName:
# GitHub repo where to put the pipeline script and project. mandatory
githubRepoUrl: https://github.com/aFlyBird0/jenkins-file-test
githubRepoUrl: https://github.com/xxx/xxx
Copy link
Member

Choose a reason for hiding this comment

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

Maybe it is better to use some placeholder like YOUR_GITHUB_ACCOUNT/YOUR_TEST_PROJECT_NAME here.

Copy link
Member Author

@aFlyBird0 aFlyBird0 Aug 8, 2022

Choose a reason for hiding this comment

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

True, but this tends to trigger bad links checks. Maybe we could add a . *YOUR.* link ignore rule?

Copy link
Member

Choose a reason for hiding this comment

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

True, but this tends to trigger bad links checks. Maybe we could add a . *YOUR.* link ignore rule?

Oh, I got it. Let me merge this pr first. And I think it's ok to add .*YOUR_.* ignore rule. @aFlyBird0

Signed-off-by: Bird <aflybird0@gmail.com>
@aFlyBird0 aFlyBird0 requested a review from a team as a code owner August 8, 2022 03:06
@aFlyBird0 aFlyBird0 requested a review from daniel-hutao August 8, 2022 03:06
@aFlyBird0 aFlyBird0 assigned daniel-hutao and unassigned aFlyBird0 Aug 8, 2022
@daniel-hutao daniel-hutao merged commit 224b0e2 into devstream-io:main Aug 8, 2022
@aFlyBird0 aFlyBird0 deleted the docs-plugin-list branch August 12, 2022 01:35
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.

4 participants