Skip to content

ci: added paths-ignore in tests GHA - #464

Merged
TorstenDittmann merged 5 commits into
appwrite:mainfrom
Ananya2001-an:ci-add-paths-ignore
Jan 2, 2024
Merged

ci: added paths-ignore in tests GHA#464
TorstenDittmann merged 5 commits into
appwrite:mainfrom
Ananya2001-an:ci-add-paths-ignore

Conversation

@Ananya2001-an

Copy link
Copy Markdown
Contributor

What does this PR do?

Added paths-ignore option for running tests GHA only when certain files are changed.

Related PRs and Issues

closes#462

Have you read the Contributing Guidelines on issues?

Yes.

@vercel

vercelBot commented Jun 19, 2023

Copy link
Copy Markdown

@Ananya2001-an is attempting to deploy a commit to the appwrite Team on Vercel.

A member of the Team first needs to authorize it.

@TGlideTGlide left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you for the PR 😄

Just one small comment

Comment thread.github/workflows/tests.yml
Comment thread.github/workflows/tests.yml Outdated
@Ananya2001-an

Copy link
Copy Markdown
ContributorAuthor

Hey @TGlide I have done the changes you can review them again :)

@TGlide

Copy link
Copy Markdown
Contributor

Hey @TGlide I have done the changes you can review them again :)

Awesome, thank you! LGTM 🎉

@TorstenDittmann
TorstenDittmann merged commit 3359937 into appwrite:mainJan 2, 2024
@Ananya2001-an
Ananya2001-an deleted the ci-add-paths-ignore branch January 3, 2024 06:34
Sign up for freeto 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.

🚀 Feature request : CI - ignore some files in event trigger of tests GHA

3 participants

@Ananya2001-an@TGlide@TorstenDittmann