Skip to content

ci: trigger ci when the configuration file of current ci changes#10382

Merged
juststillthinking merged 2 commits into
masterfrom
doc_ci_path
Oct 25, 2023
Merged

ci: trigger ci when the configuration file of current ci changes#10382
juststillthinking merged 2 commits into
masterfrom
doc_ci_path

Conversation

@jiangfucheng

Copy link
Copy Markdown
Member

Description

Now, for docs relevant ci will be triggered only when the docs have changed. But if the ci configuration file changes, it will not execute the ci. Like this PR, I can't ensure the change is well if the ci not be executed.

Fixes # (issue)

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

Comment thread .github/workflows/link-check.yml Outdated
Comment thread .github/workflows/link-check.yml Outdated
@shreemaan-abhishek

Copy link
Copy Markdown
Contributor

Great catch!

@juststillthinking
juststillthinking merged commit fb007ca into master Oct 25, 2023
@jiangfucheng
jiangfucheng deleted the doc_ci_path branch October 25, 2023 10:48
hongbinhsu pushed a commit to fitphp/apix that referenced this pull request Nov 1, 2023
* upstream/master: (83 commits)
  fix: make install failed on mac (apache#10403)
  feat(zipkin): add variable (apache#10361)
  test(clickhouse-logger): to show that different endpoints will be chosen randomly (apache#8777)
  chore(deps): bump actions/setup-node from 3.8.1 to 4.0.0 (apache#10381)
  ci: fix the grpc test error (apache#10388)
  ci: trigger ci when doc-lint.yml changes (apache#10382)
  docs: fix usage of incorrect default admin api port (apache#10391)
  feat: Add authorization params to openid-connect plugin (apache#10058)
  feat: integrate authz-keycloak with secrets resource (apache#10353)
  fix(traffic-split): post_arg match fails because content-type contains charset (apache#10372)
  fix(consul): worker will not exit while reload or quit (apache#10342)
  chore: update rules for unresponded issues (apache#10354)
  docs: Update APISIX usecases in README (apache#10358)
  test: use http2 to test limit-req plugin (apache#10334)
  test: use http2 to test limit-conn plugin (apache#10332)
  chore: remove stream_proxy.only in config-default.yaml (apache#10337)
  docs: update underscore to hyphen in HTTP headers in `response-rewrite` plugin (apache#10347)
  fix: typos in comments (apache#10330)
  feat: support config stream_route upstream in service (apache#10298)
  fix: keep healthcheck target state when upstream changes (apache#10312)
  ...
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