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

fix lint and introduce a link checker workflow #387

Merged
merged 7 commits into from
Nov 10, 2022

Conversation

tianleh
Copy link
Member

@tianleh tianleh commented Nov 9, 2022

Signed-off-by: Tianle Huang tianleh@amazon.com

Description

Discovered failed commits while working on another task. Will later check how to enforce such.

Test

The workflow can detect when there is lint issue.
https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions/runs/3432476065/jobs/5721790748

After fixing the link, the workflow is successful. https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions/runs/3432488571/jobs/5721817064

Issues Resolved

Resolves #360

Check List

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Tianle Huang <tianleh@amazon.com>
@tianleh tianleh requested a review from a team as a code owner November 9, 2022 18:24
Signed-off-by: Tianle Huang <tianleh@amazon.com>
Signed-off-by: Tianle Huang <tianleh@amazon.com>
Signed-off-by: Tianle Huang <tianleh@amazon.com>
Signed-off-by: Tianle Huang <tianleh@amazon.com>
Signed-off-by: Tianle Huang <tianleh@amazon.com>
Signed-off-by: Tianle Huang <tianleh@amazon.com>
@tianleh tianleh changed the title fix lint fix lint and introduce a link checker workflow Nov 9, 2022
Copy link
Member

@ashwin-pc ashwin-pc left a comment

Choose a reason for hiding this comment

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

Thank you! I no longer have to skip the git pre commit check

@tianleh tianleh merged commit 20bcdaf into opensearch-project:main Nov 10, 2022
opensearch-trigger-bot bot pushed a commit that referenced this pull request Nov 10, 2022
* fix lint

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* run lint

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* add .lycheeexclude

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* update workflow

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* fix

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* test fail

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* update

Signed-off-by: Tianle Huang <tianleh@amazon.com>

Signed-off-by: Tianle Huang <tianleh@amazon.com>
(cherry picked from commit 20bcdaf)
opensearch-trigger-bot bot pushed a commit that referenced this pull request Nov 10, 2022
* fix lint

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* run lint

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* add .lycheeexclude

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* update workflow

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* fix

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* test fail

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* update

Signed-off-by: Tianle Huang <tianleh@amazon.com>

Signed-off-by: Tianle Huang <tianleh@amazon.com>
(cherry picked from commit 20bcdaf)
tianleh added a commit that referenced this pull request Nov 10, 2022
* fix lint

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* run lint

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* add .lycheeexclude

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* update workflow

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* fix

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* test fail

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* update

Signed-off-by: Tianle Huang <tianleh@amazon.com>

Signed-off-by: Tianle Huang <tianleh@amazon.com>
(cherry picked from commit 20bcdaf)

Co-authored-by: Tianle Huang <60111637+tianleh@users.noreply.github.com>
seraphjiang pushed a commit that referenced this pull request Nov 10, 2022
* fix lint

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* run lint

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* add .lycheeexclude

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* update workflow

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* fix

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* test fail

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* update

Signed-off-by: Tianle Huang <tianleh@amazon.com>

Signed-off-by: Tianle Huang <tianleh@amazon.com>
(cherry picked from commit 20bcdaf)

Co-authored-by: Tianle Huang <60111637+tianleh@users.noreply.github.com>
jakubp-eliatra pushed a commit to sebastianmichalski/opensearch-dashboards-functional-test that referenced this pull request Mar 24, 2023
* fix lint

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* run lint

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* add .lycheeexclude

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* update workflow

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* fix

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* test fail

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* update

Signed-off-by: Tianle Huang <tianleh@amazon.com>

Signed-off-by: Tianle Huang <tianleh@amazon.com>
Signed-off-by: Jakub Przybylski <jakub.przybylski@eliatra.com>
leanneeliatra pushed a commit to leanneeliatra/opensearch-dashboards-functional-test-fork that referenced this pull request Sep 15, 2023
… (opensearch-project#391)

* fix lint

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* run lint

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* add .lycheeexclude

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* update workflow

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* fix

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* test fail

Signed-off-by: Tianle Huang <tianleh@amazon.com>

* update

Signed-off-by: Tianle Huang <tianleh@amazon.com>

Signed-off-by: Tianle Huang <tianleh@amazon.com>
(cherry picked from commit 20bcdaf)

Co-authored-by: Tianle Huang <60111637+tianleh@users.noreply.github.com>
Signed-off-by: leanne.laceybyrne@eliatra.com <leanne.laceybyrne@eliatra.com>
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.

[PROPOSAL] Add Lint check to CI
4 participants